[PATCH] D77627: [WebAssembly][MC] Fix leak of std::string members in MCSymbolWasm

Dan Gohman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 7 10:19:00 PDT 2020


sunfish accepted this revision.
sunfish added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyAsmPrinter.h:50
+    return Names.back().get();
+  }
+
----------------
Can this function be private in WebAssemblyAsmPrinter?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77627/new/

https://reviews.llvm.org/D77627





More information about the llvm-commits mailing list