[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