Page MenuHomePhabricator

[WebAssembly] Support Leak Sanitizer on Emscripten

Authored by quantum on Jun 3 2019, 4:10 PM.



LSan is currently being ported to Emscripten and mostly works.

Enabling the support in upstream would simplify testing.

Diff Detail


Event Timeline

quantum created this revision.Jun 3 2019, 4:10 PM
aheejin accepted this revision.Jun 3 2019, 5:22 PM
This revision is now accepted and ready to land.Jun 3 2019, 5:22 PM

Oh, can we have a test case?

LSan is supposed to be linked in to replace some C library functions, and does not change the generated LLVM IR code, so there is nothing to actually test in LLVM.

I see. Do you want me to commit this, or you are gonna get a commit access?

It's not urgent, so I am willing to wait a day or two for commit access. If I still can't get access by then, I'll ask someone else to commit this.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptJun 5 2019, 6:37 PM