From d86d3fe0c33bea37595084129cea46ab2241d33c Mon Sep 17 00:00:00 2001
From: Evan Cheng <evan.cheng@apple.com>
Date: Sun, 31 Jan 2010 00:59:31 +0000
Subject: Do not mark no-return calls tail calls. It'll screw up special calls
 like longjmp and it doesn't make much sense for performance reason. If my
 logic is faulty, please let me know.

llvm-svn: 94937
---
 llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'llvm/lib/Transforms')

diff --git a/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp b/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
index 162d902cfa4..913dd73cc17 100644
--- a/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
+++ b/llvm/lib/Transforms/Scalar/TailRecursionElimination.cpp
@@ -184,10 +184,11 @@ bool TailCallElim::runOnFunction(Function &F) {
   if (!FunctionContainsEscapingAllocas)
     for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB)
       for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ++I)
-        if (CallInst *CI = dyn_cast<CallInst>(I)) {
-          CI->setTailCall();
-          MadeChange = true;
-        }
+        if (CallInst *CI = dyn_cast<CallInst>(I))
+          if (!CI->doesNotReturn()) {
+            CI->setTailCall();
+            MadeChange = true;
+          }
 
   return MadeChange;
 }
-- 
cgit v1.2.3