[llvm] [memprof][NFC] Free symbolizer memory eagerly (PR #75849)
Teresa Johnson via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 18 13:43:47 PST 2023
================
@@ -344,7 +344,7 @@ Error RawMemProfReader::initialize(std::unique_ptr<MemoryBuffer> DataBuffer) {
Object, std::move(Context), /*UntagAddresses=*/false);
if (!SOFOr)
return report(SOFOr.takeError(), FileName);
- Symbolizer = std::move(SOFOr.get());
+ auto Symbolizer = std::move(SOFOr.get());
----------------
teresajohnson wrote:
Yeah that's a good idea. In fact, we can move it after setupForSymbolization(), although I don't think there is a memory savings from doing that.
https://github.com/llvm/llvm-project/pull/75849
More information about the llvm-commits
mailing list