Since we're not checking for the more general AllocationInst first, we need to explic...

that Object is an Argument before casting it to one.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47268 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Analysis/BasicAliasAnalysis.cpp b/lib/Analysis/BasicAliasAnalysis.cpp
index 5124376a98a0df9c511128bce315dce081b47f60..fa5c5b894c81a4bcf99bfbdcc5ee1466a5f54ec5 100644 (file)
--- a/lib/Analysis/BasicAliasAnalysis.cpp
+++ b/lib/Analysis/BasicAliasAnalysis.cpp
@@ -272,7 +272,7 @@ 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<AllocaInst>(Object) ||
-          cast<Argument>(Object)->hasByValAttr())
+          (isa<Argument>(Object) && cast<Argument>(Object)->hasByValAttr()))
         if (CallInst *CI = dyn_cast<CallInst>(CS.getInstruction()))
           if (CI->isTailCall())
             return NoModRef;