[llvm-commits] [llvm] r47220 - in /llvm/trunk: lib/Transforms/IPO/GlobalOpt.cpp test/Transforms/GlobalOpt/2008-02-16-NestAttr.ll

[Chris Lattner]

On Feb 18, 2008, at 9:35 AM, Duncan Sands wrote:

> Hi Chris,
>
>>>> Nice.  Out of curiousity, how does nest do to codegen?
>>>
>>> 'nest' causes a specific register to be grabbed for the
>>> parameter in calls.  So removing it doesn't do much :)
>>
>> Ok.
>
> if you want me to remove the transform, just ask :)  I mostly did it
> because seeing all those useless "nest" attributes floating around
> annoys me when rummaging about in Ada produced .ll's.

I have no problem with it.  It makes sense to me.

>> You can just do:
>>   CallSite User(cast<Instruction>(*UI));
>>
>> and then handle user generically.
>
> Actually you couldn't, but you can now (see the CallSite changes just
> committed).


Ah, strange, I thought that was allowed :)

-Chris