This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: optimization/7147: ifcvt.c problem (regression)

From: Franz Sirl <Franz dot Sirl-kernel at lauterbach dot com>
To: Richard Henderson <rth at redhat dot com>
Cc: gcc-gnats at gcc dot gnu dot org,gcc-patches at gcc dot gnu dot org,Mark Mitchell <mark at codesourcery dot com>
Date: Thu, 18 Jul 2002 15:05:16 +0200
Subject: Re: optimization/7147: ifcvt.c problem (regression)
References: <200206302240.20224@enzo.bigblue.local><20020627225516.16958.qmail@sources.redhat.com><200206302240.20224@enzo.bigblue.local>

At 23:27 16.07.2002, Richard Henderson wrote:

This looks like fallout from

2002-05-03  Richard Henderson  <rth@redhat.com>

        PR opt/6534
        * ifcvt.c (noce_try_store_flag, noce_try_store_flag_constants,
        noce_try_store_flag_inc, noce_try_store_flag_mask, noce_try_cmove,
        noce_try_cmove_arith, noce_try_minmax, noce_try_abs): Insert new
        code before JUMP, not EARLIEST.

I'm testing the following on x86.  Give it a whirl on ppc as well?


r~


        * ifcvt.c (noce_get_condition): Make certain that the condition
        is valid at JUMP.

That fixes the small executable testcase I posted and inspection of the assembly shows that the original mozilla source code is fine too now. Bootstrap and regression testing on powerpc-linux-gnu without problems.

Franz.

Follow-Ups:
- Re: optimization/7147: ifcvt.c problem (regression)
  - From: Mark Mitchell

References:
- Re: optimization/7147: ifcvt.c problem (regression)
  - From: Richard Henderson

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]