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: [Bug target/32437] [4.3 Regression] MIPS: FAIL in gcc.dg/cleanup-[8|9|10|11].c


Richard Sandiford <richard@codesourcery.com> writes:
> Kenneth Zadeck <zadeck@naturalbridge.com> writes:
>> 2007-06-23  Kenneth Zadeck <zadeck@naturalbridge.com>
>>
>>     PR middle-end/32437
>>     *dce.c (deletable_insn_p): Add extra parameter and recurse if insn
>>     is a PARALLEL.
>>     (prescan_insns_for_dce): Add extra parameter.
>
> Kenny found that this patch introduced problems on x86 (I think it was)
> because it applied the special handling for bare CLOBBERs to those
> inside PARALLELs as well.  We don't want that; bare USEs and CLOBBERs
> are special DF markers, but USEs and CLOBBERs inside PARALLELs are parts
> of asms or define_insns.
>
> Kenny pre-approved the patch below.  Bootstrapped & regression-tested
> on x86_64-linux-gnu.  Applied to mainline.

This patch apparently caused 32475 to resurface, so I reverted it.

Richard


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