Linker should not remap null operands of metadata
authorVictor Hernandez <vhernandez@apple.com>
Sat, 6 Feb 2010 01:31:55 +0000 (01:31 +0000)
committerVictor Hernandez <vhernandez@apple.com>
Sat, 6 Feb 2010 01:31:55 +0000 (01:31 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@95468 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Linker/LinkModules.cpp

index 279c4bec357ff78514003624069d6e9279c6ca14..7f441b0a471516fdd6936f31335c4e99ee6a60d1 100644 (file)
@@ -395,8 +395,12 @@ static Value *RemapOperand(const Value *In,
   } else if (const MDNode *MD = dyn_cast<MDNode>(In)) {
     if (MD->isFunctionLocal()) {
       SmallVector<Value*, 4> Elts;
-      for (unsigned i = 0, e = MD->getNumOperands(); i != e; ++i)
-        Elts.push_back(RemapOperand(MD->getOperand(i), ValueMap));
+      for (unsigned i = 0, e = MD->getNumOperands(); i != e; ++i) {
+        if (MD->getOperand(i))
+          Elts.push_back(RemapOperand(MD->getOperand(i), ValueMap));
+        else
+          Elts.push_back(NULL);
+      }
       Result = MDNode::get(In->getContext(), Elts.data(), MD->getNumOperands());
     } else {
       Result = const_cast<Value*>(In);