This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
PATCH: Remove the extra VEX check
- From: "H.J. Lu" <hongjiu dot lu at intel dot com>
- To: binutils at sourceware dot org
- Date: Fri, 4 May 2012 11:08:45 -0700
- Subject: PATCH: Remove the extra VEX check
- Reply-to: "H.J. Lu" <hjl dot tools at gmail dot com>
Hi,
There is
if (!i.tm.opcode_modifier.vex)
{
}
right before
if (i.tm.opcode_modifier.vex)
I checked in this patch to replace the second VEX check with else.
H.J.
---
Index: ChangeLog
===================================================================
RCS file: /cvs/src/src/gas/ChangeLog,v
retrieving revision 1.4715
diff -u -p -r1.4715 ChangeLog
--- ChangeLog 4 May 2012 17:55:38 -0000 1.4715
+++ ChangeLog 4 May 2012 18:05:25 -0000
@@ -1,5 +1,9 @@
2012-05-04 H.J. Lu <hongjiu.lu@intel.com>
+ * config/tc-i386.c (output_insn): Remove the extra VEX check.
+
+2012-05-04 H.J. Lu <hongjiu.lu@intel.com>
+
* config/tc-i386.c (match_template): Improve unsupported error
message.
Index: config/tc-i386.c
===================================================================
RCS file: /cvs/src/src/gas/config/tc-i386.c,v
retrieving revision 1.486
diff -u -p -r1.486 tc-i386.c
--- config/tc-i386.c 4 May 2012 17:55:38 -0000 1.486
+++ config/tc-i386.c 4 May 2012 18:05:26 -0000
@@ -6257,8 +6257,7 @@ check_prefix:
if (*q)
FRAG_APPEND_1_CHAR (*q);
}
-
- if (i.tm.opcode_modifier.vex)
+ else
{
for (j = 0, q = i.prefix; j < ARRAY_SIZE (i.prefix); j++, q++)
if (*q)