Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-20 Thread Bin.Cheng
> > Cc:GCC Patches > > Subject:Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap > > cases. > > > > > > > > On Sun, Nov 11, 2018 at 9:02 AM bin.cheng > > > wrote: > > >> > > >> Hi, > > >> T

Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-19 Thread Christophe Lyon
On Wed, 14 Nov 2018 at 11:10, bin.cheng wrote: > > -- > Sender:Richard Biener > Sent at:2018 Nov 13 (Tue) 23:03 > To:bin.cheng > Cc:GCC Patches > Subject:Re: [PATCH PR84648]Adjust loop exit conditions for loo

Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-14 Thread Richard Biener
On Wed, Nov 14, 2018 at 11:09 AM bin.cheng wrote: > > -- > Sender:Richard Biener > Sent at:2018 Nov 13 (Tue) 23:03 > To:bin.cheng > Cc:GCC Patches > Subject:Re: [PATCH PR84648]Adjust loop exit condition

Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-14 Thread bin.cheng
-- Sender:Richard Biener Sent at:2018 Nov 13 (Tue) 23:03 To:bin.cheng Cc:GCC Patches Subject:Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases. > > On Sun, Nov 11, 2018 at 9:02 AM bin.cheng

Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-13 Thread Richard Biener
On Sun, Nov 11, 2018 at 9:02 AM bin.cheng wrote: > > Hi, > This patch fixes PR84648 by adjusting exit conditions for loop-until-wrap > cases. > It only handles simple cases in which IV.base are constants because we rely on > current niter analyzer which doesn't handle parameterized bound in wrapp

Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-11 Thread Bin.Cheng
On Sun, Nov 11, 2018 at 7:20 PM Bernhard Reutner-Fischer wrote: > > On Sun, 11 Nov 2018 16:02:33 +0800 > "bin.cheng" wrote: > > Quick observation unrelated to the real patch. > > I think the coding style mandates to use the type itself and not the > underlying structure. > > I know there are alot

Re: [PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-11 Thread Bernhard Reutner-Fischer
On Sun, 11 Nov 2018 16:02:33 +0800 "bin.cheng" wrote: Quick observation unrelated to the real patch. I think the coding style mandates to use the type itself and not the underlying structure. I know there are alot of places from before our switch to C++ that still use enum tree_code or enum mac

[PATCH PR84648]Adjust loop exit conditions for loop-until-wrap cases.

2018-11-11 Thread bin.cheng
Hi, This patch fixes PR84648 by adjusting exit conditions for loop-until-wrap cases. It only handles simple cases in which IV.base are constants because we rely on current niter analyzer which doesn't handle parameterized bound in wrapped case. It could be relaxed in the future. Bootstrap and tes