------- Comment #21 from richard at codesourcery dot com 2007-06-30 12:26 ------- Subject: Re: [4.3 Regression] MIPS: FAIL in gcc.dg/cleanup-[8|9|10|11].c
Richard Sandiford <[EMAIL PROTECTED]> writes: > Kenneth Zadeck <[EMAIL PROTECTED]> writes: >> 2007-06-23 Kenneth Zadeck <[EMAIL PROTECTED]> >> >> 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 -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32437
