Re: divmod transform: add test-cases

2016-08-15 Thread Jeff Law
On 08/12/2016 01:17 AM, Richard Biener wrote: Note that for the main patch I don't like the current state of the divmod libcall issue. I think we need to solve this in a more reasonable manner and not expose this oddness to a GIMPLE level pass. I haven't looked at the main patch at all.

Re: divmod transform: add test-cases

2016-08-12 Thread Prathamesh Kulkarni
On 12 August 2016 at 12:47, Richard Biener wrote: > On Thu, 11 Aug 2016, Jeff Law wrote: > >> On 08/09/2016 04:58 AM, Prathamesh Kulkarni wrote: >> > ping https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01869.html >> This seems to be dependent upon other patches, this is not OK

Re: divmod transform: add test-cases

2016-08-12 Thread Richard Biener
On Thu, 11 Aug 2016, Jeff Law wrote: > On 08/09/2016 04:58 AM, Prathamesh Kulkarni wrote: > > ping https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01869.html > This seems to be dependent upon other patches, this is not OK until all > prereqs are resolved. > > You're using SI/DI in the

Re: divmod transform: add test-cases

2016-08-11 Thread Jeff Law
On 08/09/2016 04:58 AM, Prathamesh Kulkarni wrote: ping https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01869.html This seems to be dependent upon other patches, this is not OK until all prereqs are resolved. You're using SI/DI in the descriptions, but then using more traditional C types like

Re: divmod transform: add test-cases

2016-08-09 Thread Prathamesh Kulkarni
ping https://gcc.gnu.org/ml/gcc-patches/2016-07/msg01869.html Thanks, Prathamesh On 28 July 2016 at 19:07, Prathamesh Kulkarni wrote: > Hi, > The following patch adds test-cases for divmod transform. > I separated the SImode tests into separate file from DImode

divmod transform: add test-cases

2016-07-28 Thread Prathamesh Kulkarni
Hi, The following patch adds test-cases for divmod transform. I separated the SImode tests into separate file from DImode tests because certain arm configs (cortex-15) have hardware div insn for SImode but not for DImode, and for that config we want SImode tests to be disabled but not DImode