This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Fix to my lenagth_attribute patch
- To: gcc-patches at gcc dot gnu dot org
- Subject: Fix to my lenagth_attribute patch
- From: Jan Hubicka <jh at suse dot cz>
- Date: Wed, 28 Jun 2000 15:09:07 +0200
Hi,
I've just installed following patch to fix two obvious problems introduced
by my previous patch. First one was reported by Grahams and second
by Urlich.. thanks.
I've decided to use "obvious bufix" rule in this case, since it
may break bootstrap in some circmuatens.
Honza
Wed Jun 28 14:46:58 MET DST 2000 Jan Hubicka <jh@suse.cz>
* i386.md (prologue_set_got): Set length_immediate field.
(testqi_ccno_1): Add missing '@' character.
Index: egcs/gcc//config/i386/i386.md
===================================================================
RCS file: /cvs/gcc/egcs/gcc/config/i386/i386.md,v
retrieving revision 1.160
diff -c -3 -p -r1.160 i386.md
*** i386.md 2000/06/27 10:24:38 1.160
--- i386.md 2000/06/28 12:47:00
***************
*** 4806,4812 ****
(match_operand:QI 1 "nonmemory_operand" "n,n,qn,n"))
(const_int 0)))]
""
! "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}"
--- 4806,4813 ----
(match_operand:QI 1 "nonmemory_operand" "n,n,qn,n"))
(const_int 0)))]
""
! "@
! 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}"
***************
*** 8452,8457 ****
--- 8453,8461 ----
return \"add{l}\\t{%1+[.-%X2], %0|%0, %a1+(.-%X2)}\";
}"
[(set_attr "type" "alu")
+ ; Since this insn may have two constant operands, we must set the
+ ; length manually.
+ (set_attr "length_immediate" "4")
(set_attr "mode" "SI")])
(define_insn "prologue_get_pc"