diff options
| author | Nick Lewycky <nicholas@mxc.ca> | 2010-07-24 19:43:25 +0000 |
|---|---|---|
| committer | Nick Lewycky <nicholas@mxc.ca> | 2010-07-24 19:43:25 +0000 |
| commit | fee12b29851983ab536d46ed0fe85bddd665638a (patch) | |
| tree | 55f549e657e6e28037b002253567ac95fbb6cf70 /lib/Transforms | |
| parent | 1ec9e6a5dccbc6042d25f03f224d351f15812aec (diff) | |
| download | external_llvm-fee12b29851983ab536d46ed0fe85bddd665638a.zip external_llvm-fee12b29851983ab536d46ed0fe85bddd665638a.tar.gz external_llvm-fee12b29851983ab536d46ed0fe85bddd665638a.tar.bz2 | |
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.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@109353 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Utils/ValueMapper.cpp | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/lib/Transforms/Utils/ValueMapper.cpp b/lib/Transforms/Utils/ValueMapper.cpp index 3f6a90c..7b60d96 100644 --- a/lib/Transforms/Utils/ValueMapper.cpp +++ b/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<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)) { |