Skip to content

Commit de1d189

Browse files
author
MarcoFalke
committed
Merge bitcoin/bitcoin#27056: doc: use arch agnostic clang path in fuzzing doc (macOS)
b49e19c doc: use arch agnostic clang path in fuzzing doc (macOS) (fanquake) Pull request description: The current path will only work for clang installed via brew on x86_64 macOS. ACKs for top commit: hebasto: ACK b49e19c, similar to 7028365. Tree-SHA512: 8ae4845e1953d5a7178f2b422e2241af1057d8cce1ab79da65df0cd068456dbf85da3489355f81fc4ee09ba602a4b53e989e2dc02476b4abf6c5b3bc3e96473b
2 parents 51d51d3 + b49e19c commit de1d189

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

doc/fuzzing.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,10 +136,10 @@ You may also need to take care of giving the correct path for `clang` and
136136
`clang++`, like `CC=/path/to/clang CXX=/path/to/clang++` if the non-systems
137137
`clang` does not come first in your path.
138138
139-
Full configure that was tested on macOS Catalina with `brew` installed `llvm`:
139+
Full configure that was tested on macOS with `brew` installed `llvm`:
140140
141141
```sh
142-
./configure --enable-fuzz --with-sanitizers=fuzzer,address,undefined CC=/usr/local/opt/llvm/bin/clang CXX=/usr/local/opt/llvm/bin/clang++ --disable-asm
142+
./configure --enable-fuzz --with-sanitizers=fuzzer,address,undefined --disable-asm CC=$(brew --prefix llvm)/bin/clang CXX=$(brew --prefix llvm)/bin/clang++
143143
```
144144
145145
Read the [libFuzzer documentation](https://llvm.org/docs/LibFuzzer.html) for more information. This [libFuzzer tutorial](https://github.com/google/fuzzing/blob/master/tutorial/libFuzzerTutorial.md) might also be of interest.

0 commit comments

Comments
 (0)