diff options
author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-04-20 20:03:59 +0000 |
---|---|---|
committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2016-04-20 20:03:59 +0000 |
commit | a2495d9c5ab976232edf677ef147d6dee63fcb8c (patch) | |
tree | 988275a1a121ad86e68ab80aae24269a5346b453 /llvm/lib/IR/DIBuilder.cpp | |
parent | a83bfeac9d919ee99ce0c99d43339114bfb0f014 (diff) | |
download | llvm-a2495d9c5ab976232edf677ef147d6dee63fcb8c.zip llvm-a2495d9c5ab976232edf677ef147d6dee63fcb8c.tar.gz llvm-a2495d9c5ab976232edf677ef147d6dee63fcb8c.tar.bz2 |
IR: Stop copying vectors of TrackingMDRef in DIBuilder
No real functionality change here, just avoiding an unnecessary copy of
std::vector<TrackingMDRef> for every subprogram with variables.
llvm-svn: 266907
Diffstat (limited to 'llvm/lib/IR/DIBuilder.cpp')
-rw-r--r-- | llvm/lib/IR/DIBuilder.cpp | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/llvm/lib/IR/DIBuilder.cpp b/llvm/lib/IR/DIBuilder.cpp index 23a822b5..690918c 100644 --- a/llvm/lib/IR/DIBuilder.cpp +++ b/llvm/lib/IR/DIBuilder.cpp @@ -97,12 +97,18 @@ void DIBuilder::finalize() { DISubprogramArray SPs = MDTuple::get(VMContext, AllSubprograms); auto resolveVariables = [&](DISubprogram *SP) { - if (MDTuple *Temp = SP->getVariables().get()) { - const auto &PV = PreservedVariables.lookup(SP); - SmallVector<Metadata *, 4> Variables(PV.begin(), PV.end()); - DINodeArray AV = getOrCreateArray(Variables); - TempMDTuple(Temp)->replaceAllUsesWith(AV.get()); - } + MDTuple *Temp = SP->getVariables().get(); + if (!Temp) + return; + + SmallVector<Metadata *, 4> Variables; + + auto PV = PreservedVariables.find(SP); + if (PV != PreservedVariables.end()) + Variables.append(PV->second.begin(), PV->second.end()); + + DINodeArray AV = getOrCreateArray(Variables); + TempMDTuple(Temp)->replaceAllUsesWith(AV.get()); }; for (auto *SP : SPs) resolveVariables(SP); |