]> granicus.if.org Git - llvm/commitdiff
XCOFFObjectWriter - silence static analyzer dyn_cast<> null dereference warning....
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 14 Oct 2019 16:46:11 +0000 (16:46 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Mon, 14 Oct 2019 16:46:11 +0000 (16:46 +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@374788 91177308-0d34-0410-b5e6-96231b3b80d8

lib/MC/XCOFFObjectWriter.cpp

index 5d8735b017d3f8eb48284974dcbe0480c2a527b6..dbea1185f90e8d96d38d8302d1850a64290fca8f 100644 (file)
@@ -218,7 +218,7 @@ void XCOFFObjectWriter::executePostLayoutBinding(
   DenseMap<const MCSectionXCOFF *, ControlSection *> WrapperMap;
 
   for (const auto &S : Asm) {
-    const MCSectionXCOFF *MCSec = dyn_cast<const MCSectionXCOFF>(&S);
+    const auto *MCSec = cast<const MCSectionXCOFF>(&S);
     assert(WrapperMap.find(MCSec) == WrapperMap.end() &&
            "Cannot add a csect twice.");