https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68277

--- Comment #5 from Oleg Endo <olegendo at gcc dot gnu.org> ---
(In reply to Kazumoto Kojima from comment #4)
> (In reply to Oleg Endo from comment #3)
> > I haven't read reload.*, but my speculation is that if something in reload
> > instantiates that rtx with an imm8 constant to calculate addresses, it might
> > also try to instantiate it with reg = reg + reg if the constant doesn't fit
> > into imm8.  Maybe we just don't hit the case (yet).  So it's probably safer
> > to allow reg = reg + reg, too.
> 
> Sounds reasonable.

Could you please add it to your nightly test run and commit it if there no
other new failures?  I'll be away for a few days...

> 
> > I guess those "new" failures are R0 related?
> 
> Yep, spill failure in R0.  Our old friend.

Hm ... so maybe for now we should enable -mlra by default and add a 2nd
postreload CSE pass as you have tried in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67732#c2 ?

Reply via email to