Revert this because we can't clone cyclic MDNodes which are creating during a
authorNick Lewycky <nicholas@mxc.ca>
Sat, 24 Jul 2010 20:54:02 +0000 (20:54 +0000)
committerNick Lewycky <nicholas@mxc.ca>
Sat, 24 Jul 2010 20:54:02 +0000 (20:54 +0000)
build of llvm-gcc.

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

lib/Transforms/Utils/ValueMapper.cpp

index 7b60d9697c6a6376ed06f81518e8a3006169417d..3f6a90c94ebb67360b0f2082409257e14f677bcb 100644 (file)
@@ -27,9 +27,10 @@ Value *llvm::MapValue(const Value *V, ValueToValueMapTy &VM) {
   // NOTE: VMSlot can be invalidated by any reference to VM, which can grow the
   // DenseMap.  This includes any recursive calls to MapValue.
 
-  // Global values do not need to be seeded into the VM if they are using
-  // the identity mapping.
-  if (isa<GlobalValue>(V) || isa<InlineAsm>(V) || isa<MDString>(V))
+  // Global values and non-function-local metadata do not need to be seeded into
+  // the VM if they are using the identity mapping.
+  if (isa<GlobalValue>(V) || isa<InlineAsm>(V) || isa<MDString>(V) ||
+      (isa<MDNode>(V) && !cast<MDNode>(V)->isFunctionLocal()))
     return VMSlot = const_cast<Value*>(V);
 
   if (const MDNode *MD = dyn_cast<MDNode>(V)) {