Whether function-local or not, a MDNode may reference a Function in which case

it needs to be mapped to refer to the function in the new module, not the old
one. Fixes PR7700.

llvm-svn: 109353
This commit is contained in:
Nick Lewycky 2010-07-24 19:43:25 +00:00
parent 6ca585fc41
commit 14b69d59dd
1 changed files with 3 additions and 4 deletions

View File

@ -27,10 +27,9 @@ 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 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()))
// 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))
return VMSlot = const_cast<Value*>(V);
if (const MDNode *MD = dyn_cast<MDNode>(V)) {