[llvm-commits] [llvm] r151429 - in /llvm/trunk: include/llvm/Transforms/IPO.h include/llvm/Transforms/IPO/InlinerPass.h include/llvm/Transforms/Utils/Cloning.h lib/Transforms/IPO/InlineAlways.cpp lib/Transforms/IPO/InlineSimple.cpp lib/Transforms/IPO/Inliner.cpp lib/Transforms/Utils/InlineFunction.cpp

Chad Rosier mcrosier at apple.com
Sun Feb 26 18:53:57 PST 2012


On Feb 26, 2012, at 4:11 PM, Eric Christopher wrote:

> 
> On Feb 24, 2012, at 6:56 PM, Chad Rosier wrote:
> 
>> +bool llvm::InlineFunction(InvokeInst *II, InlineFunctionInfo &IFI, bool InsertLifetime) {
> 
> FWIW instead of InsertLifetime perhaps a bool of Optimize instead? In case there were other things we wanted to do/avoid doing at inlining time for optimized code versus non-optimized.

I though the same thing initially, but then decided on the InsertLifetime as it's more concise.  If this bool starts controlling other features I figure we could make the suggested change at that time.

 Chad

> *shrug*
> 
> -eric

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120226/6b330e81/attachment.html>


More information about the llvm-commits mailing list