[PATCH, committed] x86: Replace AddrPrefixOp0 with AddrPrefixOpReg

Jan Beulich jbeulich@suse.com
Wed May 9 06:29:00 GMT 2018


>>> "H.J. Lu" <hongjiu.lu@intel.com> 05/07/18 7:01 PM >>>
>--- a/gas/config/tc-i386.c
>+++ b/gas/config/tc-i386.c
>@@ -5997,14 +5997,18 @@ process_suffix (void)
>/* Now select between word & dword operations via the operand
>size prefix, except for instructions that will ignore this
>prefix anyway.  */
>-      if (i.tm.opcode_modifier.addrprefixop0)
>+      if (i.reg_operands > 0
>+	  && i.types[0].bitfield.reg
>+	  && i.tm.opcode_modifier.addrprefixopreg
>+	  && (i.tm.opcode_modifier.immext
>+	      || i.operands == 1))

What is this comparison against 1 for? All the single operand SVM insns
have ImmExt set.

Jan





More information about the Binutils mailing list