Re: Committed: Fix sh regression for target/55160

2012-11-01 Thread Kaz Kojima
Oleg Endo wrote: > I'd like to add a test case for this. > > Attached patch was tested with > make -k check-gcc RUNTESTFLAGS="sh.exp=pr55160.c --target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > OK to install? OK. Regards, kaz

Re: Committed: Fix sh regression for target/55160

2012-11-01 Thread Joern Rennecke
Quoting Oleg Endo : I'm using: /* { dg-skip-if "" { "sh*-*-*" } { "-m5*"} { "" } } */ Sorry, I mentally mixed up the arguments.

Re: Committed: Fix sh regression for target/55160

2012-11-01 Thread Oleg Endo
On Thu, 2012-11-01 at 05:18 -0400, Joern Rennecke wrote: > Quoting Oleg Endo : > > > I'd like to add a test case for this. > > > > Attached patch was tested with > > make -k check-gcc RUNTESTFLAGS="sh.exp=pr55160.c --target_board=sh-sim > > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/

Re: Committed: Fix sh regression for target/55160

2012-11-01 Thread Joern Rennecke
Quoting Oleg Endo : I'd like to add a test case for this. Attached patch was tested with make -k check-gcc RUNTESTFLAGS="sh.exp=pr55160.c --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" OK to install? That'll be for the SH maintainers to decide. However,

Re: Committed: Fix sh regression for target/55160

2012-11-01 Thread Oleg Endo
On Thu, 2012-11-01 at 02:31 -0400, Joern Rennecke wrote: > gen_doloop_end_split creates a pattern that sets pc, hence emit_jump_insn > has to be used instead of emit_insn. > > Committed as obvious. I'd like to add a test case for this. Attached patch was tested with make -k check-gcc RUNTESTFLAG

Committed: Fix sh regression for target/55160

2012-10-31 Thread Joern Rennecke
gen_doloop_end_split creates a pattern that sets pc, hence emit_jump_insn has to be used instead of emit_insn. Committed as obvious. 2012-11-01 Joern Rennecke PR target/55160 * config/sh/sh.md (doloop_end): Use emit_jump_insn. Index: config/sh/sh.md ==