aeubanks updated this revision to Diff 379111.
aeubanks added a comment.
refactor
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111582/new/
https://reviews.llvm.org/D111582
Files:
clang/lib/CodeGen/BackendUtil.cpp
Index: clang/lib/CodeGen/Backe
dblaikie added a comment.
Any chance of refactoring this code so naturally these objects are lexically
scoped to their necessary lifetime, and just cease to exist before the code gen
passes are run? (looks like this function is pretty long and might benefit from
being broken up into pieces anyw
aeubanks created this revision.
Herald added a subscriber: ormris.
aeubanks requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This saves a tiny bit of memory in non-LTO builds [1].
[1]
https://llvm-compile-time-tracker.com/compare.php?from=