x86 patch i386.md fix to eliminate gas warning.

Graham Stott grahams@redhat.com
Tue Oct 17 11:35:00 GMT 2000


Hi

This patch eliminates the "Warning: using `%esi' instead of `%si' due to `l' suffix"
diagnostic generated by gas when compiling bfd/elf.c

We were generating "testl $1,%si" rather than "testl $1,%esi" for the 4th alternative.

OK to commit ?

Graham

Changelog

	* i386.md (testqi_1): Add missing operand prefix for operand 0.

------------------------------------------------------------------------
Index: i386.md
===================================================================
RCS file: /cvs/gcc/egcs/gcc/config/i386/i386.md,v
retrieving revision 1.185
diff -c -p -r1.185 i386.md
*** i386.md     2000/10/16 16:09:38     1.185
--- i386.md     2000/10/17 15:39:11
***************
*** 5370,5376 ****
     test{b}\\t{%1, %0|%0, %1}
     test{b}\\t{%1, %0|%0, %1}
     test{b}\\t{%1, %0|%0, %1}
!    test{l}\\t{%1, %0|%0, %1}"
    [(set_attr "type" "test")
     (set_attr "modrm" "0,1,1,1")
     (set_attr "mode" "QI,QI,QI,SI")
--- 5370,5376 ----
     test{b}\\t{%1, %0|%0, %1}
     test{b}\\t{%1, %0|%0, %1}
     test{b}\\t{%1, %0|%0, %1}
!    test{l}\\t{%1, %k0|%k0, %1}"
    [(set_attr "type" "test")
     (set_attr "modrm" "0,1,1,1")
     (set_attr "mode" "QI,QI,QI,SI")
-------------------------------------------------------------------------


More information about the Gcc-patches mailing list