Re: [PATCH] [SH] Fix target/48596

2012-03-11 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch moves it as suggested to gcc.c-torture/compile. > Briefly tested by running the gcc.c-torture/compile set on sh-him > -m4a-single -ml. You forgot to remove two dg-* lines: > +/* { dg-do compile } */ > +/* { dg-options "-O1" } */ unneeded for this gcc.c-tort

Re: [PATCH] [SH] Fix target/48596

2012-03-11 Thread Oleg Endo
On Tue, 2012-03-06 at 08:24 +0900, Kaz Kojima wrote: > Oleg Endo wrote: > > I'd like to add the test case from the PR to the testsuite. > > > > Tested with > > make check-gcc RUNTESTFLAGS="sh.exp=pr48596.c --target_board=sh-sim > > \{-m2/-ml,-m2/-mb,-m2a-single/-mb, > > -m4-single/-ml,-m4-single/

Re: [PATCH] [SH] Fix target/48596

2012-03-05 Thread Kaz Kojima
Oleg Endo wrote: > I'd like to add the test case from the PR to the testsuite. > > Tested with > make check-gcc RUNTESTFLAGS="sh.exp=pr48596.c --target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a-single/-mb, > -m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" > > OK? A gcc.c-torture/com

Re: [PATCH] [SH] Fix target/48596

2012-03-05 Thread Oleg Endo
On Thu, 2012-03-01 at 13:11 +0900, Kaz Kojima wrote: > Hi, > > The attached patch is to avoid PR target/48596 which is a 4.7 > regression on SH. > [...] I'd like to add the test case from the PR to the testsuite. Tested with make check-gcc RUNTESTFLAGS="sh.exp=pr48596.c --target_board=sh-sim \

Re: [PATCH] [SH] Fix target/48596

2012-03-01 Thread Kaz Kojima
Oleg Endo wrote: > BTW, the patch also seems to fix PR 48806. Ah, I've confirmed it. Thanks for pointing it out. Regards, kaz

Re: [PATCH] [SH] Fix target/48596

2012-03-01 Thread Oleg Endo
On Thu, 2012-03-01 at 13:11 +0900, Kaz Kojima wrote: > Hi, > > The attached patch is to avoid PR target/48596 which is a 4.7 > regression on SH. It seems that now IRA aggressively tried to > use fp regs as the holder for memory addresses on this port. > SH requires a special fpul register to move

Re: [PATCH] [SH] Fix target/48596

2012-03-01 Thread Mike Stump
On Feb 29, 2012, at 8:11 PM, Kaz Kojima wrote: > The attached patch is to avoid PR target/48596 which is a 4.7 > regression on SH. It seems that now IRA aggressively tried to > use fp regs as the holder for memory addresses on this port. Gosh, I hope IRA is cleaned up, as other ports do have weir

[PATCH] [SH] Fix target/48596

2012-02-29 Thread Kaz Kojima
Hi, The attached patch is to avoid PR target/48596 which is a 4.7 regression on SH. It seems that now IRA aggressively tried to use fp regs as the holder for memory addresses on this port. SH requires a special fpul register to move from the fp regs to the general regs which are legitimate for po