This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Bug in combine: Triple combine fails due to shared rtlsubstitution
- From: "Hartmut Penner" <HPENNER at de dot ibm dot com>
- To: gcc-patches at gcc dot gnu dot org
- Date: Fri, 11 Jan 2002 13:43:57 +0100
- Subject: Re: PATCH: Bug in combine: Triple combine fails due to shared rtlsubstitution
Andreas Jaeger
<aj@suse.de> To: Hartmut Penner/Germany/IBM@IBMDE
Sent by: cc: gcc-patches@gcc.gnu.org
aj@suse.de Subject: Re: PATCH: Bug in combine: Triple combine fails due to shared rtl substitution
01/11/02 10:05
AM
>"Hartmut Penner" <HPENNER@de.ibm.com> writes:
>> Index: combine.c
>> ===================================================================
>> RCS file: /cvs/gcc/gcc/gcc/combine.c,v
>> retrieving revision 1.254
>> diff -u -r1.254 combine.c
>> --- combine.c 2002/01/10 07:37:51 1.254
>> +++ combine.c 2002/01/11 08:23:13
>> @@ -9519,8 +9558,7 @@
>> x = new;
>> else
>> {
>> - if (x == 0 || GET_CODE (x) != code || GET_MODE (x) != shift_mode)
>> - x = gen_rtx_fmt_ee (code, shift_mode, varop, const_rtx);
>> + x = gen_rtx_fmt_ee (code, shift_mode, varop, const_rtx);
>>
>> SUBST (XEXP (x, 0), varop);
>> SUBST (XEXP (x, 1), const_rtx);
>>
>> OK to commit?
>Has this test been bootstrapped and regtested? On which targets?
Yes, done it on s390x-ibm-linux-gnu plattform.
To confirm, that there will not be regression, it can bootstrap it also on
ia86.
>Please tell this information as requestion on
http://gcc.gnu.org/contribute.html
>Andreas
--
> Andreas Jaeger
> SuSE Labs aj@suse.de
> private aj@arthur.inka.de
> http://www.suse.de/~aj