[llvm-commits] [llvm] r69631 - /llvm/trunk/lib/Transforms/Scalar/LoopUnroll.cpp

[Chris Lattner]

On Apr 20, 2009, at 3:32 PM, Evan Cheng wrote:
>> Well... That's not entirely true. Each GEP can also become a several
>> instructions. Is there a test case to demonstrate the merit of this
>> change?
>>
>
> How about changing it to what InlineCost.cpp does?

Better yet, change both to use if GEP->hasAllConstantIndices()  :)

-Chris