This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug target/32437] [4.3 Regression] MIPS: FAIL in gcc.dg/cleanup-[8|9|10|11].c
- From: "richard dot guenther at gmail dot com" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: 23 Jun 2007 16:12:06 -0000
- Subject: [Bug target/32437] [4.3 Regression] MIPS: FAIL in gcc.dg/cleanup-[8|9|10|11].c
- References: <bug-32437-7151@http.gcc.gnu.org/bugzilla/>
- Reply-to: gcc-bugzilla at gcc dot gnu dot org
------- Comment #10 from richard dot guenther at gmail dot com 2007-06-23 16:12 -------
Subject: Re: [4.3 Regression] MIPS: FAIL in gcc.dg/cleanup-[8|9|10|11].c
On 6/23/07, Kenneth Zadeck <zadeck@naturalbridge.com> wrote:
> This patch changes dce:deletable_insn_p so that it looks at all of the
> top level
> clauses in a parallel to make it's decision. It was not keeping insns
> that had a top level USE or UNSPEC if they were inside of parallels.
>
> This should fix pr32437 and perhaps other things.
>
> The patch has only been tested on ppc and x86-64. It is harmless on
> those platforms.
> It is likely to make a difference on pa-risc and mips where there are
> parallels that contain top level unspecs.
>
> Ok to commit?
This is ok for mainline.
Thanks,
Richard.
>
> 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.
>
>
>
>
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32437