Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-07 Thread Jeff Law
On 12/07/2016 02:24 PM, Jakub Jelinek wrote: On Wed, Dec 07, 2016 at 02:18:15PM -0700, Jeff Law wrote: On 12/05/2016 02:14 PM, Jakub Jelinek wrote: Hi! As shown on the testcase, with K&R definitions and fn prototypes with promoted types, we can end up computing caller's value ranges in wider t

Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-07 Thread Jakub Jelinek
On Wed, Dec 07, 2016 at 02:18:15PM -0700, Jeff Law wrote: > On 12/05/2016 02:14 PM, Jakub Jelinek wrote: > >Hi! > > > >As shown on the testcase, with K&R definitions and fn prototypes with > >promoted types, we can end up computing caller's value ranges in wider > >type than the parameter actually

Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-07 Thread Jeff Law
On 12/05/2016 02:14 PM, Jakub Jelinek wrote: Hi! As shown on the testcase, with K&R definitions and fn prototypes with promoted types, we can end up computing caller's value ranges in wider type than the parameter actually has in the function. The problem with that is that wide_int_storage::from

Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-06 Thread Prathamesh Kulkarni
On 6 December 2016 at 14:50, Richard Biener wrote: > On Tue, 6 Dec 2016, Jakub Jelinek wrote: > >> On Tue, Dec 06, 2016 at 09:36:55AM +0100, Richard Biener wrote: >> > > As shown on the testcase, with K&R definitions and fn prototypes with >> > > promoted types, we can end up computing caller's va

Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-06 Thread Richard Biener
On Tue, 6 Dec 2016, Jakub Jelinek wrote: > On Tue, Dec 06, 2016 at 09:36:55AM +0100, Richard Biener wrote: > > > As shown on the testcase, with K&R definitions and fn prototypes with > > > promoted types, we can end up computing caller's value ranges in wider > > > type than the parameter actually

Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-06 Thread Jakub Jelinek
On Tue, Dec 06, 2016 at 09:36:55AM +0100, Richard Biener wrote: > > As shown on the testcase, with K&R definitions and fn prototypes with > > promoted types, we can end up computing caller's value ranges in wider > > type than the parameter actually has in the function. > > The problem with that is

Re: [PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-06 Thread Richard Biener
On Mon, 5 Dec 2016, Jakub Jelinek wrote: > Hi! > > As shown on the testcase, with K&R definitions and fn prototypes with > promoted types, we can end up computing caller's value ranges in wider > type than the parameter actually has in the function. > The problem with that is that wide_int_storag

[PATCH] Fix ICE due to IPA-VRP (PR tree-optimization/78681)

2016-12-05 Thread Jakub Jelinek
Hi! As shown on the testcase, with K&R definitions and fn prototypes with promoted types, we can end up computing caller's value ranges in wider type than the parameter actually has in the function. The problem with that is that wide_int_storage::from can actually wrap around, so either as in the