Page MenuHomePhabricator

[WebAssembly] Support Leak Sanitizer on Emscripten
ClosedPublic

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

Details

Summary

LSan is currently being ported to Emscripten and mostly works.

Enabling the support in upstream would simplify testing.

Diff Detail

Repository
rL LLVM

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