public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] trivial fix for OPTION_MAVXSCALAR define in tc-i386.c
@ 2010-06-03 16:31 Quentin Neill
  2010-06-03 16:39 ` H.J. Lu
  0 siblings, 1 reply; 3+ messages in thread
From: Quentin Neill @ 2010-06-03 16:31 UTC (permalink / raw)
  To: binutils

[-- Attachment #1: Type: text/plain, Size: 162 bytes --]

Trivial fix for #define OPTION_MAVXSCALAR to follow other defines.

gas:
    * config/tc-i386.c fix OPTION_MAVXSCALAR define.

Okay to commit?

-- 
Quentin Neill

[-- Attachment #2: 7010_option_define.diff.txt --]
[-- Type: text/plain, Size: 636 bytes --]

gas:
    * config/tc-i386.c fix OPTION_MAVXSCALAR define.

Index: tc-i386.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-i386.c,v
retrieving revision 1.435
diff -u -d -u -r1.435 tc-i386.c
--- tc-i386.c	22 Apr 2010 03:10:48 -0000	1.435
+++ tc-i386.c	3 Jun 2010 16:12:32 -0000
@@ -8048,7 +8048,7 @@
 #define OPTION_MOLD_GCC (OPTION_MD_BASE + 9)
 #define OPTION_MSSE2AVX (OPTION_MD_BASE + 10)
 #define OPTION_MSSE_CHECK (OPTION_MD_BASE + 11)
-#define OPTION_MAVXSCALAR (OPTION_MSSE_CHECK + 11)
+#define OPTION_MAVXSCALAR (OPTION_MD_BASE + 12)
 
 struct option md_longopts[] =
 {

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] trivial fix for OPTION_MAVXSCALAR define in tc-i386.c
  2010-06-03 16:31 [PATCH] trivial fix for OPTION_MAVXSCALAR define in tc-i386.c Quentin Neill
@ 2010-06-03 16:39 ` H.J. Lu
  2010-06-03 17:01   ` Sebastian Pop
  0 siblings, 1 reply; 3+ messages in thread
From: H.J. Lu @ 2010-06-03 16:39 UTC (permalink / raw)
  To: Quentin Neill; +Cc: binutils

On Thu, Jun 3, 2010 at 9:31 AM, Quentin Neill
<quentin.neill.gnu@gmail.com> wrote:
> Trivial fix for #define OPTION_MAVXSCALAR to follow other defines.
>
> gas:
>    * config/tc-i386.c fix OPTION_MAVXSCALAR define.

Please change it to "* config/tc-i386.c (OPTION_MAVXSCALAR): Fix define."
OK with this change.

Thanks.

H.J.
---
>
> Okay to commit?
>
> --
> Quentin Neill
>



-- 
H.J.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] trivial fix for OPTION_MAVXSCALAR define in tc-i386.c
  2010-06-03 16:39 ` H.J. Lu
@ 2010-06-03 17:01   ` Sebastian Pop
  0 siblings, 0 replies; 3+ messages in thread
From: Sebastian Pop @ 2010-06-03 17:01 UTC (permalink / raw)
  To: H.J. Lu; +Cc: Quentin Neill, binutils

On Thu, Jun 3, 2010 at 11:38, H.J. Lu <hjl.tools@gmail.com> wrote:
> On Thu, Jun 3, 2010 at 9:31 AM, Quentin Neill
> <quentin.neill.gnu@gmail.com> wrote:
>> Trivial fix for #define OPTION_MAVXSCALAR to follow other defines.
>>
>> gas:
>>    * config/tc-i386.c fix OPTION_MAVXSCALAR define.
>
> Please change it to "* config/tc-i386.c (OPTION_MAVXSCALAR): Fix define."
> OK with this change.

Committed.

Sebastian

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2010-06-03 17:01 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-03 16:31 [PATCH] trivial fix for OPTION_MAVXSCALAR define in tc-i386.c Quentin Neill
2010-06-03 16:39 ` H.J. Lu
2010-06-03 17:01   ` Sebastian Pop

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).