]> granicus.if.org Git - llvm/commitdiff
CrossDSOCFI - silence static analyzer dyn_cast<> null dereference warning. NFCI.
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 21 Oct 2019 17:15:25 +0000 (17:15 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 21 Oct 2019 17:15:25 +0000 (17:15 +0000)
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

index 5f3d0c2e3733c2ae14e31388a06e7ce7f102fb75..e20159ba0db59bbe627a174ddcffce00660bc34e 100644 (file)
@@ -104,7 +104,7 @@ void CrossDSOCFI::buildCFICheck(Module &M) {
   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();