This revision was automatically updated to reflect the committed changes.
Closed by commit rC328384: [libFuzzer] Use OptForFuzzing attribute with
-fsanitize=fuzzer. (authored by morehouse, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D44798?vs=139673&id=139675#toc
Reposito
morehouse updated this revision to Diff 139673.
morehouse added a comment.
- Remove new substitutions. Use -O0 to avoid optimization.
https://reviews.llvm.org/D44798
Files:
clang/lib/CodeGen/CodeGenFunction.cpp
compiler-rt/test/fuzzer/SimpleCmpTest.cpp
compiler-rt/test/fuzzer/SwapCmpTest
vitalybuka accepted this revision.
vitalybuka added inline comments.
This revision is now accepted and ready to land.
Comment at: compiler-rt/test/fuzzer/lit.cfg:88
+
+config.substitutions.append(('%no_opt_cpp_compiler',
+ generate_compiler_cmd(is_cpp=True, fuzzer_enabled=Tr
morehouse added a comment.
16-bit variables are switched to 32-bit variables in SwapCmpTest.cpp and
SimpleCmpTest.cpp. This is because those tests rely on libFuzzer's TraceCMP
heuristic to pass, but 16-bit compares are not considered for the heuristic.
The only reason the test used to pass was
morehouse created this revision.
morehouse added reviewers: kcc, vitalybuka.
Disables certain CMP optimizations to improve fuzzing signal under -O1
and -O2.
Switches all fuzzer tests to -O2 except for a few leak tests where the
leak is optimized out under -O2.
https://reviews.llvm.org/D44798
F
