[llvm-commits] [llvm] r49299 - /llvm/trunk/lib/Transforms/Utils/InlineFunction.cpp

[Duncan Sands]

Hi,

> Mark calls to llvm.stacksave, llvm.stackrestore as
> nounwind.  When such calls are inlined into something
> else that is invoked, they were getting changed to invokes,
> which is badness.

>      CallInst *SavedPtr = CallInst::Create(StackSave, "savedstack", 
>                                            FirstNewBlock->begin());

if you get hold of an intrinsic using Intrinsic::getDeclaration then
it is automagically marked nounwind.  Is there any reason not to use
Intrinsic::getDeclaration here?

Ciao,

Duncan.