[PATCH] D35443: Enable 64-bit Darwin LeakSanitizer by default on AddressSanitizer builds

Francis Ricci via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 14 16:43:36 PDT 2017


fjricci added a comment.

> Please wait on enabling this on-by-default until we perform some more testing on our side.

You will probably want to wait until https://reviews.llvm.org/D35432 goes in, as that has the potential to introduce false positives (although I didn't see any in my testing). This patch is blocked on that anyway.

> Have you tried e.g. running this on apps and programs written in Swift?

I don't have access to much in the way of Swift code, but I can try adding it to the test suite and see how that goes.

> Does bootstrap'd LLVM/Clang report no leaks?

Didn't think to run the clang test suite, working on that now.


https://reviews.llvm.org/D35443





More information about the llvm-commits mailing list