[llvm-commits] ConstantRange

[Chris Lattner]

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.

-Chris