[PATCH] D62830: [WebAssembly] Support Leak Sanitizer on Emscripten

[Guanzhong Chen via Phabricator via cfe-commits]

quantum created this revision.
quantum added reviewers: tlively, aheejin.
Herald added subscribers: cfe-commits, sunfish, jgravelle-google, sbc100, dschuff.
Herald added a project: clang.

LSan is currently being ported to Emscripten and mostly works.

Enabling the support in upstream would simplify testing.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D62830

Files:
  clang/lib/Driver/ToolChains/WebAssembly.cpp


Index: clang/lib/Driver/ToolChains/WebAssembly.cpp
===================================================================
--- clang/lib/Driver/ToolChains/WebAssembly.cpp
+++ clang/lib/Driver/ToolChains/WebAssembly.cpp
@@ -211,7 +211,7 @@
 SanitizerMask WebAssembly::getSupportedSanitizers() const {
   SanitizerMask Res = ToolChain::getSupportedSanitizers();
   if (getTriple().isOSEmscripten()) {
-    Res |= SanitizerKind::Vptr;
+    Res |= SanitizerKind::Vptr | SanitizerKind::Leak;
   }
   return Res;
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62830.202817.patch
Type: text/x-patch
Size: 506 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190603/d02455a2/attachment.bin>