From 14b69d59dd5f2dec92ed68738fe1fe8a4c6fc312 Mon Sep 17 00:00:00 2001 From: Nick Lewycky Date: Sat, 24 Jul 2010 19:43:25 +0000 Subject: 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 --- llvm/lib/Transforms/Utils/ValueMapper.cpp | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'llvm/lib/Transforms/Utils') diff --git a/llvm/lib/Transforms/Utils/ValueMapper.cpp b/llvm/lib/Transforms/Utils/ValueMapper.cpp index 3f6a90c94eb..7b60d9697c6 100644 --- a/llvm/lib/Transforms/Utils/ValueMapper.cpp +++ b/llvm/lib/Transforms/Utils/ValueMapper.cpp @@ -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(V) || isa(V) || isa(V) || - (isa(V) && !cast(V)->isFunctionLocal())) + // Global values do not need to be seeded into the VM if they are using + // the identity mapping. + if (isa(V) || isa(V) || isa(V)) return VMSlot = const_cast(V); if (const MDNode *MD = dyn_cast(V)) { -- cgit v1.2.3