Re: [PATCH] Add value_range_base (w/o equivs)

2018-11-12 Thread Aldy Hernandez
Ug ok. On Mon, Nov 12, 2018, 12:10 Richard Biener On Mon, 12 Nov 2018, Aldy Hernandez wrote: > > > > > > > On 11/11/18 3:53 AM, Richard Biener wrote: > > > On Fri, 9 Nov 2018, Aldy Hernandez wrote: > > > > > > > On 11/9/18 9:19 AM, Richard Biener wrote: > > > > > > > > > > This adds

Re: [PATCH] Add value_range_base (w/o equivs)

2018-11-12 Thread Richard Biener
On Mon, 12 Nov 2018, Aldy Hernandez wrote: > > > On 11/11/18 3:53 AM, Richard Biener wrote: > > On Fri, 9 Nov 2018, Aldy Hernandez wrote: > > > > > On 11/9/18 9:19 AM, Richard Biener wrote: > > > > > > > > This adds value_range_base, a base class of class value_range > > > > with all members

Re: [PATCH] Add value_range_base (w/o equivs)

2018-11-12 Thread Aldy Hernandez
On 11/11/18 3:53 AM, Richard Biener wrote: On Fri, 9 Nov 2018, Aldy Hernandez wrote: On 11/9/18 9:19 AM, Richard Biener wrote: This adds value_range_base, a base class of class value_range with all members but the m_equiv one. First of all, thanks so much for doing this! I have

Re: [PATCH] Add value_range_base (w/o equivs)

2018-11-11 Thread Richard Biener
On Fri, 9 Nov 2018, Aldy Hernandez wrote: > On 11/9/18 9:19 AM, Richard Biener wrote: > > > > This adds value_range_base, a base class of class value_range > > with all members but the m_equiv one. > > First of all, thanks so much for doing this! > > > > > I have looked into the sole GC user,

Re: [PATCH] Add value_range_base (w/o equivs)

2018-11-09 Thread Jeff Law
On 11/9/18 7:19 AM, Richard Biener wrote: > > This adds value_range_base, a base class of class value_range > with all members but the m_equiv one. > > I have looked into the sole GC user, IPA propagation, and replaced > the value_range use there with value_range_base since it also > asserted

Re: [PATCH] Add value_range_base (w/o equivs)

2018-11-09 Thread Aldy Hernandez
On 11/9/18 9:19 AM, Richard Biener wrote: This adds value_range_base, a base class of class value_range with all members but the m_equiv one. First of all, thanks so much for doing this! I have looked into the sole GC user, IPA propagation, and replaced the value_range use there with

[PATCH] Add value_range_base (w/o equivs)

2018-11-09 Thread Richard Biener
This adds value_range_base, a base class of class value_range with all members but the m_equiv one. I have looked into the sole GC user, IPA propagation, and replaced the value_range use there with value_range_base since it also asserted the equiv member is always NULL. This in turn means I