This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [SH] Adding some peepholes (PR 61142)
- From: Oleg Endo <oleg dot endo at t-online dot de>
- To: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 26 Feb 2015 20:16:05 +0100
- Subject: Re: [SH] Adding some peepholes (PR 61142)
- Authentication-results: sourceware.org; auth=none
- References: <1424900120 dot 14981 dot 407 dot camel at yam-132-YW-E178-FTW> <20150226 dot 192151 dot 149980256 dot kkojima at rr dot iij4u dot or dot jp>
On Thu, 2015-02-26 at 19:21 +0900, Kaz Kojima wrote:
> Oleg Endo <oleg.endo@t-online.de> wrote:
> > These are the peepholes as mentioned in PR 65153 and in PR 61142. They
> > try to wallpaper some bad RA choices and reduce the CSiBE code size by
> > approx. 3.9K bytes.
> >
> > A problem I ran into with this one is that the peephole2 pass drops
> > REG_INC notes, which makes the following passes produce garbage
> > sometimes. Instead of rejecting automodify mems in the peephole2
> > patterns, for now I'm manually adding the REG_INC notes after emitting
> > move insns. Maybe peephole2 could do that automatically in the future.
> >
> > Tested with
> > make -k check RUNTESTFLAGS="--target_board=sh-sim
> > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}".
> >
> > Kaz, could you also please pre-test this on sh4-linux?
>
> No new failures on trunk revision 220928.
Thanks. Committed as r221026.
Cheers,
Oleg