On Wed, 18 Sep 2013, Richard Earnshaw wrote: > On 16/09/13 15:13, Richard Biener wrote: > > +void > > +get_range_info (tree name, double_int &min, double_int &max, > > + enum value_range_type &range_type) > > > > I'm not sure we want to use references. Well - first time. > > Personally, I don't think we should ever allow non-const references. > Use of references means you can't tell from the source code that an > argument to a function can be modified. It leads to confusion and makes > things harder to debug.
That's a good argument, so please make get_range_info take pointers to min/max/range_type. Richard.