This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] fix target/44606, reload bug on SPE
- From: Nathan Froyd <froydnj at codesourcery dot com>
- To: Eric Botcazou <ebotcazou at adacore dot com>
- Cc: gcc-patches at gcc dot gnu dot org, bernds at codesourcery dot com
- Date: Fri, 1 Oct 2010 09:33:50 -0700
- Subject: Re: [PATCH] fix target/44606, reload bug on SPE
- References: <20100930182927.GL32503@codesourcery.com> <201010010016.50677.ebotcazou@adacore.com>
On Fri, Oct 01, 2010 at 12:16:50AM +0200, Eric Botcazou wrote:
> > (reload_as_needed): Delete insn if emit_reload_insns returns
> > true.
> > (choose_reload_regs): Save equiv in reload_override_in for
> > output reloads. Set reg_rtx from reload_override_in.
>
> I think that deleting new insns in reload is too risky on the branches. Can't
> we tighten the above condition instead on the branches so that it will return
> false in this case?
I don't see a good way to tighten the above condition. Deleting the
cleverness is an option, though. ;)
The other possibility is to teach postreload about this sort of thing
and have it delete the second insn.
-Nathan