Re: [SH] PR 50751 - Add support for SH2A movu.b and movu.w insns

2012-08-14 Thread Kaz Kojima
Oleg Endo wrote: > This adds support for the SH2A instructions movu.b and movu.w for > zero-extending mem loads with displacement addressing. > Tested on rev 190332 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

[SH] PR 50751 - Add support for SH2A movu.b and movu.w insns

2012-08-13 Thread Oleg Endo
Hello, This adds support for the SH2A instructions movu.b and movu.w for zero-extending mem loads with displacement addressing. Tested on rev 190332 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? Chee

Re: [SH] PR 50751

2012-08-09 Thread Kaz Kojima
Oleg Endo wrote: > This patch fixes a minor issue related to the displacement addressing > patterns, which leads to useless movt exts.* sequences and one of the > predicates wrongly accepting non-mem ops. > > Tested on rev 190151 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-

[SH] PR 50751

2012-08-08 Thread Oleg Endo
Hello, This patch fixes a minor issue related to the displacement addressing patterns, which leads to useless movt exts.* sequences and one of the predicates wrongly accepting non-mem ops. Tested on rev 190151 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/

Re: [SH] PR 50751- add HImode testcases

2012-04-11 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch adds some more test cases for PR 50751. > > Tested on sh-sim with > make check-gcc RUNTESTFLAGS="sh.exp=pr50751* --target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a-single/-mb,-m4-single/-ml, > -m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" > > to confirm that

[SH] PR 50751- add HImode testcases

2012-04-11 Thread Oleg Endo
Hi, The attached patch adds some more test cases for PR 50751. Tested on sh-sim with make check-gcc RUNTESTFLAGS="sh.exp=pr50751* --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a-single/-mb,-m4-single/-ml, -m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" to confirm that the tests pass as expected.

Re: [SH] PR 50751 - add HImode displacement addressing support

2012-04-11 Thread Oleg Endo
On Wed, 2012-04-11 at 22:10 +0900, Kaz Kojima wrote: > Oleg Endo wrote: > >> BTW, do you have the numbers of CSiBE with this? > >> > > > > Only for "-m4-single -ml -O2 -mpretend-cmove" so far. > > Not so spectacular :T > > I'll also do a comparison of more variants to see if something went > > r

Re: [SH] PR 50751 - add HImode displacement addressing support

2012-04-11 Thread Kaz Kojima
Oleg Endo wrote: >> BTW, do you have the numbers of CSiBE with this? >> > > Only for "-m4-single -ml -O2 -mpretend-cmove" so far. > Not so spectacular :T > I'll also do a comparison of more variants to see if something went > really bad. It's a bit difficult to isolate the degradations because

Re: [SH] PR 50751 - add HImode displacement addressing support

2012-04-10 Thread Oleg Endo
On Tue, 2012-04-10 at 07:14 -0700, Nathan Froyd wrote: > - Original Message - > > > BTW, do you have the numbers of CSiBE with this? > > > > Only for "-m4-single -ml -O2 -mpretend-cmove" so far. > > Not so spectacular :T > > I'll also do a comparison of more variants to see if something we

Re: [SH] PR 50751 - add HImode displacement addressing support

2012-04-10 Thread Nathan Froyd
- Original Message - > > BTW, do you have the numbers of CSiBE with this? > > Only for "-m4-single -ml -O2 -mpretend-cmove" so far. > Not so spectacular :T > I'll also do a comparison of more variants to see if something went > really bad. It's a bit difficult to isolate the degradations

Re: [SH] PR 50751 - add HImode displacement addressing support

2012-04-10 Thread Oleg Endo
On Tue, 2012-04-10 at 22:42 +0900, Kaz Kojima wrote: > Oleg Endo wrote: > > The attached patch adds HImode addressing support. > > Tested against rev. 186243 with > > sudo make -k check RUNTESTFLAGS="--target_board=sh-sim > > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb, > > -m4-sing

Re: [SH] PR 50751 - add HImode displacement addressing support

2012-04-10 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch adds HImode addressing support. > Tested against rev. 186243 with > sudo 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

[SH] PR 50751 - add HImode displacement addressing support

2012-04-09 Thread Oleg Endo
Hello, The attached patch adds HImode addressing support. Tested against rev. 186243 with sudo 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. Test ca

Re: [SH] PR 50751 - rework displacement calculations

2012-03-27 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch generalizes the move insn displacement calculations a > little bit. Before, the same address rebasing code was present in > sh_legitimize_address as well as sh_legitimize_reload_address. I've > pulled those out into a separate function as a preparation step

[SH] PR 50751 - rework displacement calculations

2012-03-26 Thread Oleg Endo
Hi, The attached patch generalizes the move insn displacement calculations a little bit. Before, the same address rebasing code was present in sh_legitimize_address as well as sh_legitimize_reload_address. I've pulled those out into a separate function as a preparation step for adding HImode dis

Re: [SH] PR 50751 - some test cases

2012-03-20 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch adds some test cases for PR 50751 to check whether > mov.b insns are generated. OK. Regards, kaz

[SH] PR 50751 - some test cases

2012-03-20 Thread Oleg Endo
Hi, The attached patch adds some test cases for PR 50751 to check whether mov.b insns are generated. Tested on sh-sim with make check-gcc RUNTESTFLAGS="sh.exp=pr50751* --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a-single/-mb,-m4-single/-ml, -m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" to con

Re: [SH] PR 50751 - Add QImode displacement addressing

2012-03-19 Thread Kaz Kojima
Oleg Endo wrote: > This is an updated version of the QImode displacement addressing patch > from the PR that applies to rev 185405. Tested on sh-sim with no new > failures. > > The issue as a whole requires some more work and I'd like to split it in > smaller incremental changes and separate pat

[SH] PR 50751 - Add QImode displacement addressing

2012-03-19 Thread Oleg Endo
Hi, This is an updated version of the QImode displacement addressing patch from the PR that applies to rev 185405. Tested on sh-sim with no new failures. The issue as a whole requires some more work and I'd like to split it in smaller incremental changes and separate patches. Cheers, Oleg Cha