lowerObjCCall - silence static analyzer dyn_cast<CallInst> null dereference warnings...
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Tue, 24 Sep 2019 10:46:30 +0000 (10:46 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Tue, 24 Sep 2019 10:46:30 +0000 (10:46 +0000)
The static analyzer is warning about a potential null dereference, but we should be able to use cast<CallInst> directly and if not assert will fire for us.

llvm-svn: 372720

llvm/lib/CodeGen/PreISelIntrinsicLowering.cpp

index 2752e18..0d2f6f9 100644 (file)
@@ -76,7 +76,7 @@ static bool lowerObjCCall(Function &F, const char *NewFn,
   }
 
   for (auto I = F.use_begin(), E = F.use_end(); I != E;) {
-    auto *CI = dyn_cast<CallInst>(I->getUser());
+    auto *CI = cast<CallInst>(I->getUser());
     assert(CI->getCalledFunction() && "Cannot lower an indirect call!");
     ++I;