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

[Duncan Sands]

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.

> 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).

Thanks for the review!

Duncan.