[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