Re: [SH] PR 39423 - Add support for SH2A movu.w insn

2012-08-21 Thread Kaz Kojima
Oleg Endo wrote: > This adds support for SH2A's movu.w insn for memory addressing cases as > described in the PR. > Tested on rev 190546 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > and no new failures. > OK? OK. R

[SH] PR 39423 - Add support for SH2A movu.w insn

2012-08-21 Thread Oleg Endo
Hello, This adds support for SH2A's movu.w insn for memory addressing cases as described in the PR. Tested on rev 190546 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK? Cheers, Oleg ChangeLog:

Re: [SH] PR 39423

2012-08-16 Thread Kaz Kojima
Oleg Endo wrote: > This fixes the issue mentioned in the PR's comment #29. > Tested on rev 190396 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > and no new failures. > OK? OK. Regards, kaz

[SH] PR 39423

2012-08-16 Thread Oleg Endo
Hello, This fixes the issue mentioned in the PR's comment #29. Tested on rev 190396 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK? Cheers, Oleg ChangeLog: PR target/39423 * config/sh

Re: [SH] PR 39423 - Fix ICEs, add more tests

2012-08-12 Thread Kaz Kojima
Oleg Endo wrote: > This patch fixes ICEs such as reported in the PR due to wrong predicate > matching and adds test cases for those. > Tested on rev 190273 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > and no new fail

[SH] PR 39423 - Fix ICEs, add more tests

2012-08-12 Thread Oleg Endo
Hello, This patch fixes ICEs such as reported in the PR due to wrong predicate matching and adds test cases for those. Tested on rev 190273 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK? Cheers, Oleg

Re: [SH] PR 39423

2012-08-09 Thread Kaz Kojima
Oleg Endo wrote: > How about the attached patch? > Is that way of dealing with the mems OK? > What could be a possible test case for the alias info issue? > > Tested on rev 190151 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-

Re: [SH] PR 39423

2012-08-08 Thread Richard Henderson
On 08/08/2012 03:12 PM, Oleg Endo wrote: > How about the attached patch? > Is that way of dealing with the mems OK? > What could be a possible test case for the alias info issue? That looks like the right sort of thing. A test case would have to be for a missed-optimization, where we failed to sc

Re: [SH] PR 39423

2012-08-08 Thread Oleg Endo
On Mon, 2012-07-30 at 08:28 -0700, Richard Henderson wrote: > On 2012-07-29 15:56, Oleg Endo wrote: > > + "&& can_create_pseudo_p ()" > > + [(set (match_dup 5) (ashift:SI (match_dup 1) (match_dup 2))) > > + (set (match_dup 6) (plus:SI (match_dup 5) (match_dup 3))) > > + (set (match_dup 0) (me

Re: [SH] PR 39423

2012-07-30 Thread Oleg Endo
On Mon, 2012-07-30 at 08:28 -0700, Richard Henderson wrote: > On 2012-07-29 15:56, Oleg Endo wrote: > > + "&& can_create_pseudo_p ()" > > + [(set (match_dup 5) (ashift:SI (match_dup 1) (match_dup 2))) > > + (set (match_dup 6) (plus:SI (match_dup 5) (match_dup 3))) > > + (set (match_dup 0) (me

Re: [SH] PR 39423

2012-07-30 Thread Richard Henderson
On 2012-07-29 15:56, Oleg Endo wrote: > + "&& can_create_pseudo_p ()" > + [(set (match_dup 5) (ashift:SI (match_dup 1) (match_dup 2))) > + (set (match_dup 6) (plus:SI (match_dup 5) (match_dup 3))) > + (set (match_dup 0) (mem:SI (plus:SI (match_dup 6) (match_dup 4] Don't create new mems l

Re: [SH] PR 39423

2012-07-29 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch adds the combine patterns as discussed in the PR. > Tested on rev 189916 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb, > -m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" >

[SH] PR 39423

2012-07-29 Thread Oleg Endo
Hello, The attached patch adds the combine patterns as discussed in the PR. Tested on rev 189916 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb, -m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" and no new failures.