This check is not correct for mallocs, so exclude them earlier.
authorOwen Anderson <resistor@mac.com>
Mon, 18 Feb 2008 09:11:02 +0000 (09:11 +0000)
committerOwen Anderson <resistor@mac.com>
Mon, 18 Feb 2008 09:11:02 +0000 (09:11 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47263 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Analysis/BasicAliasAnalysis.cpp

index c4153b937182ef2272de2608aea15bc20eb8b21e..5124376a98a0df9c511128bce315dce081b47f60 100644 (file)
@@ -271,10 +271,10 @@ BasicAliasAnalysis::getModRefInfo(CallSite CS, Value *P, unsigned Size) {
 
       // If this is a tail call and P points to a stack location, we know that
       // the tail call cannot access or modify the local stack.
-      if (isa<AllocationInst>(Object) ||
+      if (isa<AllocaInst>(Object) ||
           cast<Argument>(Object)->hasByValAttr())
         if (CallInst *CI = dyn_cast<CallInst>(CS.getInstruction()))
-          if (CI->isTailCall() && !isa<MallocInst>(Object))
+          if (CI->isTailCall())
             return NoModRef;
     }
   }