Correct over-zealous removal of hack.
Some code want to check that *any* call within a function has the 'returns
twice' attribute, not just that the current function has one.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@142221 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Analysis/InlineCost.cpp b/lib/Analysis/InlineCost.cpp
index 65db33c..40ac9a2 100644
--- a/lib/Analysis/InlineCost.cpp
+++ b/lib/Analysis/InlineCost.cpp
@@ -229,7 +229,7 @@
// _setjmp), never inline it. This is a hack because we depend on the user
// marking their local variables as volatile if they are live across a setjmp
// call, and they probably won't do this in callers.
- callsSetJmp = F->hasFnAttr(Attribute::ReturnsTwice);
+ callsSetJmp = F->callsFunctionThatReturnsTwice();
// Look at the size of the callee.
for (Function::const_iterator BB = F->begin(), E = F->end(); BB != E; ++BB)