[PATCH] D96045: [llvm-dwarfdump][locstats] Unify handling of inlined vars with no loc
Jeremy Morse via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 5 05:08:29 PST 2021
jmorse added a comment.
Thinking out loud, and motivated by memory pressure: for the per-inlining-site record, that doesn't need be be maintained before or after we visit the inlining site, right? We might be able to get away with keeping a "Current scopes inlined variables" vector on the stack and passing it down through collectStatsRecursive. That'll automagically release the vector when we're done with the scope.
llvm-dwarfdump --statistics already takes a while to examine a stage2reldeb clang binary, it'd be best to avoid allocating more global data than we need.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96045/new/
https://reviews.llvm.org/D96045
More information about the llvm-commits
mailing list