Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-05 Thread Markus Trippelsdorf
On 2016.08.03 at 15:47 +0200, Richard Biener wrote: > On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka wrote: > > VRP currently has functionality to eliminate case labels that lie > > completely outside of the switch operand's value range. This patch > > complements this functionality by teaching VR

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-04 Thread Richard Biener
On Thu, Aug 4, 2016 at 2:14 PM, Patrick Palka wrote: > On Thu, 4 Aug 2016, Richard Biener wrote: > >> On Thu, Aug 4, 2016 at 4:30 AM, Patrick Palka wrote: >> > On Wed, 3 Aug 2016, David Malcolm wrote: >> > >> >> On Wed, 2016-08-03 at 15:47 +0200, Richard Biener wrote: >> >> > On Wed, Aug 3, 2016

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-04 Thread Patrick Palka
On Thu, 4 Aug 2016, Richard Biener wrote: > On Thu, Aug 4, 2016 at 4:30 AM, Patrick Palka wrote: > > On Wed, 3 Aug 2016, David Malcolm wrote: > > > >> On Wed, 2016-08-03 at 15:47 +0200, Richard Biener wrote: > >> > On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka > >> > wrote: > >> > > VRP currentl

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-04 Thread Richard Biener
On Thu, Aug 4, 2016 at 4:30 AM, Patrick Palka wrote: > On Wed, 3 Aug 2016, David Malcolm wrote: > >> On Wed, 2016-08-03 at 15:47 +0200, Richard Biener wrote: >> > On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka >> > wrote: >> > > VRP currently has functionality to eliminate case labels that lie >>

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-03 Thread Patrick Palka
On Wed, 3 Aug 2016, David Malcolm wrote: > On Wed, 2016-08-03 at 15:47 +0200, Richard Biener wrote: > > On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka > > wrote: > > > VRP currently has functionality to eliminate case labels that lie > > > completely outside of the switch operand's value range. T

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-03 Thread Jeff Law
On 08/03/2016 09:29 AM, David Malcolm wrote: On Wed, 2016-08-03 at 15:47 +0200, Richard Biener wrote: On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka wrote: VRP currently has functionality to eliminate case labels that lie completely outside of the switch operand's value range. This patch compl

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-03 Thread David Malcolm
On Wed, 2016-08-03 at 15:47 +0200, Richard Biener wrote: > On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka > wrote: > > VRP currently has functionality to eliminate case labels that lie > > completely outside of the switch operand's value range. This patch > > complements this functionality by teac

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-03 Thread Jeff Law
On 08/03/2016 07:47 AM, Richard Biener wrote: On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka wrote: VRP currently has functionality to eliminate case labels that lie completely outside of the switch operand's value range. This patch complements this functionality by teaching VRP to also truncat

Re: [PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-03 Thread Richard Biener
On Wed, Aug 3, 2016 at 6:00 AM, Patrick Palka wrote: > VRP currently has functionality to eliminate case labels that lie > completely outside of the switch operand's value range. This patch > complements this functionality by teaching VRP to also truncate the case > label ranges that partially ov

[PATCH] Teach VRP to truncate the case ranges of a switch

2016-08-02 Thread Patrick Palka
VRP currently has functionality to eliminate case labels that lie completely outside of the switch operand's value range. This patch complements this functionality by teaching VRP to also truncate the case label ranges that partially overlap with the operand's value range. Bootstrapped and regtes