diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-02-17 00:02:27 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2015-02-17 00:02:27 +0000 |
| commit | b4749379299e852f7d35d054ddeadd71d6dc695f (patch) | |
| tree | 1a2f19f82d3c6899d8ee75577aa1bc8ba1251ef1 /llvm/unittests/ADT/ArrayRefTest.cpp | |
| parent | b0553e21ccd4d34cf53875adacd5dd502614f7c2 (diff) | |
| download | llvm-b4749379299e852f7d35d054ddeadd71d6dc695f.zip llvm-b4749379299e852f7d35d054ddeadd71d6dc695f.tar.gz llvm-b4749379299e852f7d35d054ddeadd71d6dc695f.tar.bz2 | |
AsmPrinter: Stop creating DebugLocs
While looking at a heap profile of a clang LTO bootstrap with -g, I
noticed that 2.2% of memory in an `llvm-lto` of clang is from calling
`DebugLoc::get()` in `collectVariableInfo()` (accounting for ~40% of
memory used for `MDLocation`s).
I suspect this was introduced by r226736, whose goal was to prevent
uniquing of `DebugLoc`s (goal achieved, if so).
There's no reason we need a `DebugLoc` here at all -- it was just being
used for (in)convenient API -- so the fix is to pass the scope and
inlined-at directly to `LexicalScopes::findInlinedScope()`.
llvm-svn: 229459
Diffstat (limited to 'llvm/unittests/ADT/ArrayRefTest.cpp')
0 files changed, 0 insertions, 0 deletions