GVNHoist - silence static analyzer dyn_cast<> null dereference warning in hasEHOrLoad...
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 21 Oct 2019 17:15:49 +0000 (17:15 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 21 Oct 2019 17:15:49 +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.

llvm-svn: 375429

llvm/lib/Transforms/Scalar/GVNHoist.cpp

index 1f01ba2..c87e414 100644 (file)
@@ -539,7 +539,7 @@ private:
 
     // Check for unsafe hoistings due to side effects.
     if (K == InsKind::Store) {
-      if (hasEHOrLoadsOnPath(NewPt, dyn_cast<MemoryDef>(U), NBBsOnAllPaths))
+      if (hasEHOrLoadsOnPath(NewPt, cast<MemoryDef>(U), NBBsOnAllPaths))
         return false;
     } else if (hasEHOnPath(NewBB, OldBB, NBBsOnAllPaths))
       return false;