[PATCH] D20977: [LibFuzzer] Fix fuzzer-oom.test on OSX.

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 13:40:37 PDT 2016


kcc added a comment.

Interesting solution! 
But wrong. :( 
Essentially, you are now not testing that the heap profile is printed, but that libFuzzer successfully bypasses the test. 
Instead of this, I suggest to make two copies of test/fuzzer-oom.test. 
One will remain as is and require linux. 
Another one will work on darwing too, but will not check for "Live Heap Allocations:"


http://reviews.llvm.org/D20977





More information about the llvm-commits mailing list