[llvm-commits] [llvm] r110493 - in /llvm/trunk: include/llvm/Support/ConstantRange.h lib/Support/ConstantRange.cpp

Chris Lattner clattner at apple.com
Fri Aug 6 22:29:50 PDT 2010


On Aug 6, 2010, at 9:53 PM, Nick Lewycky wrote:

> Owen Anderson wrote:
>> Author: resistor
>> Date: Fri Aug  6 19:42:06 2010
>> New Revision: 110493
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=110493&view=rev
>> Log:
>> Add a convenience constructor.
> 
> I'm highly suspicious of this. Does Support depend on VMCore elsewhere?

No... except for ConstantRange.cpp.  It looks like it already has the layering violation.  Owen, can you clean up this backwards dependency while you're in there?

-Chris



More information about the llvm-commits mailing list