[PATCH] fix target/44606, reload bug on SPE

Nathan Froyd froydnj@codesourcery.com
Fri Oct 1 16:34:00 GMT 2010


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



More information about the Gcc-patches mailing list