On Sun, 2015-09-27 at 21:03 +0900, Oleg Endo wrote:
> On Wed, 2015-09-23 at 21:04 +0900, Oleg Endo wrote:
> > Hi,
> >
> > The attached patch fixes PR 67391. Some additional reg overlapping were
> > added to the addsi3 patterns while making LRA on SH work, but not all of
> > them seem to be good. Removing them, seems to be working just fine.
> > Tested on sh-elf (LRA enabled) with make -k check
> > RUNTESTFLAGS="--target_board=sh-sim
> > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"
> > and by Kaz on sh4-linux.
> >
> > Committed to trunk as r228046 and to the GCC 5 branch as r228047.
>
> This has opened a small can of worms. The follow up patch is attached
> and has been commited to trunk as r228176. Tested by me on sh-elf and
> by Kaz on sh4-linux with LRA enabled/disabled.
>
> As a positive side effect, we get some code size reduction on the CSiBE
> set: 3345527 bytes -> 3334351 bytes -11176 bytes / -0.334058 %
>
> However, this is only when LRA is disabled because of some problems with
> LRA and its usage/handling of addsi3 insns.
>
> Backport to GCC 5 branch will follow.
Now also committed to GCC 5 branch as r228201. Tested on the branch as
above.
Cheers,
Oleg
