commit b0db20ce62f8ecfe79fcb328008aed4473e2d793
author Simon Pilgrim <llvm-dev@redking.me.uk> Mon Oct 21 17:15:25 2019 +0000
committer Simon Pilgrim <llvm-dev@redking.me.uk> Mon Oct 21 17:15:25 2019 +0000
tree 4a687cced263345bb925cc1ac160ef4bf89907e0
parent 72157fef87a5bb4aa1578cf75877ec6020a3b44f

CrossDSOCFI - silence static analyzer dyn_cast<> null dereference warning. NFCI.

The static analyzer is warning about a potential null dereference, but we should be able to use cast<> directly and if not assert will fire for us.

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

lib/Transforms/IPO/CrossDSOCFI.cpp

diff --git a/lib/Transforms/IPO/CrossDSOCFI.cpp b/lib/Transforms/IPO/CrossDSOCFI.cpp
index 5f3d0c2..e20159b 100644
--- a/lib/Transforms/IPO/CrossDSOCFI.cpp
+++ b/lib/Transforms/IPO/CrossDSOCFI.cpp
@@ -104,7 +104,7 @@
   FunctionCallee C = M.getOrInsertFunction(
       "__cfi_check", Type::getVoidTy(Ctx), Type::getInt64Ty(Ctx),
       Type::getInt8PtrTy(Ctx), Type::getInt8PtrTy(Ctx));
-  Function *F = dyn_cast<Function>(C.getCallee());
+  Function *F = cast<Function>(C.getCallee());
   // Take over the existing function. The frontend emits a weak stub so that the
   // linker knows about the symbol; this pass replaces the function body.
   F->deleteBody();