Re: [SH] PR 49468 - Integer SI abs

2011-09-28 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch and ChangeLog below should fix it. > I have also added a test case for SI mode abs. Thanks! I've applied your patch as revision 179320. Regards, kaz

Re: [SH] PR 49468 - Integer SI abs

2011-09-27 Thread Oleg Endo
On Tue, 2011-09-27 at 22:36 +0900, Kaz Kojima wrote: > Thanks for this work! A few minor style issues: > Thanks for checking and sorry for the trouble. The attached patch and ChangeLog below should fix it. I have also added a test case for SI mode abs. Cheers, Oleg ChangeLog: 2011-09-28 O

Re: [SH] PR 49468 - Integer SI abs

2011-09-27 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch improves the generated code for integer abs > operations on SH, in particular SH4. There was already some code that > was supposed to utilize SH's conditional execution it but it was never > triggered, because the standard branch-free abs code was generated at

[SH] PR 49468 - Integer SI abs

2011-09-25 Thread Oleg Endo
Hello, The attached patch improves the generated code for integer abs operations on SH, in particular SH4. There was already some code that was supposed to utilize SH's conditional execution it but it was never triggered, because the standard branch-free abs code was generated at a very early sta