]> granicus.if.org Git - llvm/commitdiff
MetadataLoader lazyLoadOneMetadata - silence static analyzer dyn_cast<MDNode> null...
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Thu, 26 Sep 2019 11:30:47 +0000 (11:30 +0000)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Thu, 26 Sep 2019 11:30:47 +0000 (11:30 +0000)
The static analyzer is warning about a potential null dereference, but we should be able to use cast<MDNode> directly and if not assert will fire for us.

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

lib/Bitcode/Reader/MetadataLoader.cpp

index fa50ec5766a75fd28f02147bba19fb1de99b426f..049164c7be70f4d4072bb48e66c57f05665ffa7f 100644 (file)
@@ -987,7 +987,7 @@ void MetadataLoader::MetadataLoaderImpl::lazyLoadOneMetadata(
   assert(ID >= MDStringRef.size() && "Unexpected lazy-loading of MDString");
   // Lookup first if the metadata hasn't already been loaded.
   if (auto *MD = MetadataList.lookup(ID)) {
-    auto *N = dyn_cast_or_null<MDNode>(MD);
+    auto *N = cast<MDNode>(MD);
     if (!N->isTemporary())
       return;
   }