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