Re: [PATCH PR71437/V2]Simplify cond with assertions in threading

2017-02-17 Thread Bin.Cheng
On Fri, Feb 17, 2017 at 3:40 AM, Jeff Law wrote: > On 02/14/2017 03:05 AM, Bin Cheng wrote: >> >> Hi, >> This is the second try fixing PR71437. The old version patch tried to fix >> issue in VRP but it requires further non-trivial change in VRP, >> specifically, to better support variable value r

Re: [PATCH PR71437/V2]Simplify cond with assertions in threading

2017-02-16 Thread Jeff Law
On 02/14/2017 03:05 AM, Bin Cheng wrote: Hi, This is the second try fixing PR71437. The old version patch tried to fix issue in VRP but it requires further non-trivial change in VRP, specifically, to better support variable value ranges. This is not appropriate at stage 4. Alternatively, th

Re: [PATCH PR71437/V2]Simplify cond with assertions in threading

2017-02-16 Thread Jeff Law
On 02/16/2017 04:55 AM, Richard Biener wrote: On Tue, Feb 14, 2017 at 11:05 AM, Bin Cheng wrote: Hi, This is the second try fixing PR71437. The old version patch tried to fix issue in VRP but it requires further non-trivial change in VRP, specifically, to better support variable value ranges

Re: [PATCH PR71437/V2]Simplify cond with assertions in threading

2017-02-16 Thread Jeff Law
On 02/14/2017 03:05 AM, Bin Cheng wrote: Hi, This is the second try fixing PR71437. The old version patch tried to fix issue in VRP but it requires further non-trivial change in VRP, specifically, to better support variable value ranges. This is not appropriate at stage 4. Alternatively, th

Re: [PATCH PR71437/V2]Simplify cond with assertions in threading

2017-02-16 Thread Richard Biener
On Tue, Feb 14, 2017 at 11:05 AM, Bin Cheng wrote: > Hi, > This is the second try fixing PR71437. The old version patch tried to fix > issue in VRP but it requires further non-trivial change in VRP, specifically, > to better support variable value ranges. This is not appropriate at stage 4. >

[PATCH PR71437/V2]Simplify cond with assertions in threading

2017-02-14 Thread Bin Cheng
Hi, This is the second try fixing PR71437. The old version patch tried to fix issue in VRP but it requires further non-trivial change in VRP, specifically, to better support variable value ranges. This is not appropriate at stage 4. Alternatively, this patch tries to fix issue by improving th