[llvm-commits] ConstantRange

[Reid Spencer]

On Wed, 2007-02-28 at 10:11 -0800, Chris Lattner wrote:
> On Feb 28, 2007, at 8:29 AM, Reid Spencer wrote:
> 
> > Chris,
> >
> > I have converted ConstantRange to use APInt internally. This  
> > simplified
> > it and got rid of some of the helper functions inside ConstantRange
> > (because they have equivalents in APInt). Now I'm thinking that
> > ConstantRange should completely avoid the use of ConstantInt in its  
> > API
> > and replace all such uses with APInt. This would mean that the class
> > could be implemented in lib/Support instead of lib/Analysis because it
> > would no longer depend on lib/VMCore.
> >
> > Is this a desirable change?
> 
> Very desirable!
> 
> > Or, do you want the interface to still work
> > with ConstantInt?
> 
> No need, the clients of ConstantRange can do the xlation.

Okay, I will make it so in increments.

> 
> -Chris