- User Since
- Jun 28 2017, 4:27 PM (55 w, 1 d)
Please watch the bots after landing this.
- Support MSan experimentally.
Benchmarks easy to discern the crash cause:
You can probably get rid of the llvm-objcopy code and make this a lot simpler with something like:
Tue, Jul 17
Maybe change the callback signature to take i, j, and UnstableIdx instead. Then we can also use IterateInline8bitCounters from UpdateObservedPCs.
I think it's fine to land. Kostya approved the previous version, and he probably won't mind the extra 0's in the initialization lists.
Ran fuzzer-test-suite's test-everything.sh with -fsanitize=memory,fuzzer and all MSan reports came from the fuzz target, not libFuzzer. I think it's time to document the new capability.
Mon, Jul 16
Try again after syncing past r337224. Hopefully that fixes your issue.
- Revert "[libFuzzer] Avoid std::cout in SimpleTest."
- Create SimpleTestStdio.cpp without STL.
After more thinking, the private libc++ probably isn't the issue here since only libFuzzer itself gets linked with that. The fuzz target is linked with whatever C++ standard library your system is configured to use.
Make sure you have the libcxx sources checked out (I think under llvm/projects). I think that should fix it.
Hmm... Seems to be crashing in std::cout <<, but nothing there should be uninitialized. Also the bots are all green.
Fri, Jul 13
The other test configs don't define config.test_exec_root at all. Do we need it?
Thu, Jul 12
Wed, Jul 11
Tue, Jul 10
Note that I can reproduce this locally by compiling with -O0.
@kcc: Do we want to just disable this test for AArch64, or should we change SetBytesForLabel to not be recursive?
Mon, Jul 9
- Correct diffbase.
- Update test headers.
Reverted due to bot breakage in r336616.
Landed in r336597.
Fri, Jul 6
Tue, Jul 3
- Remove global from FuzzerDriver.cpp.
- Remove unnecessary scope.
- Unpoison DataCopy.
- Add strlen test.
- Repeat puts().
- Merge branch 'memfuzz' into memfuzz2
- Sync with parent patch.
- Added use-after-dtor test.
- Disable/enable MSan interceptors via scoped class.
- Add _scoped_ to function names.
- Add test.
Fri, Jun 29
Thu, Jun 28
Does the bot config also explain the SIGUSR test flakiness?
Wed, Jun 27
I think arcanist was just acting up... Actual committed changes: https://reviews.llvm.org/rCRT328384
We want to keep -O2.
Does that machine build libFuzzer differently? Maybe the version of libc++ it uses is different.
This makes the test more complex. What is the reason for this change?