Skip to content

Commit 8b7c2ee

Browse files
steadmongitster
authored andcommitted
Makefile: correct example fuzz build
The comment explaining how to build the fuzzers was broken in 927c77e ("Makefile: use FUZZ_CXXFLAGS for linking fuzzers", 2018-11-14). When building fuzzers, all .c files must be compiled with coverage tracing enabled. This is not possible when using only FUZZ_CXXFLAGS, as that flag is only applied to the fuzzers themselves. Switching back to CFLAGS fixes the issue. Signed-off-by: Josh Steadmon <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent d2b86fb commit 8b7c2ee

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3104,7 +3104,7 @@ cover_db_html: cover_db
31043104
# An example command to build against libFuzzer from LLVM 4.0.0:
31053105
#
31063106
# make CC=clang CXX=clang++ \
3107-
# FUZZ_CXXFLAGS="-fsanitize-coverage=trace-pc-guard -fsanitize=address" \
3107+
# CFLAGS="-fsanitize-coverage=trace-pc-guard -fsanitize=address" \
31083108
# LIB_FUZZING_ENGINE=/usr/lib/llvm-4.0/lib/libFuzzer.a \
31093109
# fuzz-all
31103110
#

0 commit comments

Comments
 (0)