[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

[Eric Christopher]

On Feb 26, 2012, at 6:53 PM, Chad Rosier wrote:

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

Sure :)

-eric

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