[PATCH] D20947: [LibFuzzer] Disable building and running LSan tests on Apple platforms because LSan is not currently supported.

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 11:08:14 PDT 2016


delcypher added a comment.

In http://reviews.llvm.org/D20947#448542, @delcypher wrote:

> @kcc:
>
> > Why do you need to disable building the tests? 
>
> >  I would expect that "REQUIRES: lsan" is enough. No?
>
>
> My original version (not uploaded to phabricator) of the patch actually didn't disable building the tests. I decided to disable building them because I thought it was to build tests that would never be executed. However if you prefer I can change the patch to always build the tests because they do build successfully.


Sorry

s/because I thought it was to build/because I thought it was odd to build/


http://reviews.llvm.org/D20947





More information about the llvm-commits mailing list