This revision was automatically updated to reflect the committed changes.
Closed by commit rG43d7320e7111: [lldb][docs] Add documentation for LLDB
fuzzers (authored by cassanova).
Changed prior to commit:
https://reviews.llvm.org/D132148?vs=453808&id=456058#toc
Repository:
rG LLVM Github Mon
JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.
LGTM modulo inline comment.
Comment at: lldb/docs/resources/fuzzing.rst:15-17
+ $ -DLLVM_USE_SANITIZER='Address'
+ $ -DLLVM_USE_SANITIZE_COVERAGE=On
+ $ -DC
cassanova updated this revision to Diff 453808.
cassanova added a comment.
Removed the full CMake invocation for the fuzzer build configuration.
Added the information on OSS Fuzz to its own section.
Changed build directory in fuzzer execution command to use a relative path.
Removed reference to
cassanova added inline comments.
Comment at: lldb/docs/resources/fuzzing.rst:12
+
+Building the LLDB fuzzers requires a build configuration that has the address
sanitizer and sanitizer coverage. This CMake invocation will configure a build
directory that can be used to build th
mib added inline comments.
Comment at: lldb/docs/resources/fuzzing.rst:28-33
+To build a fuzzer, run the desired ninja command for the fuzzer(s) you want to
build:
+
+::
+ $ ninja lldb-target-fuzzer
+ $ ninja lldb-commandinterpreter-fuzzer
+ $ ninja lldb-expression-fuzzer
JDevlieghere added inline comments.
Comment at: lldb/docs/resources/fuzzing.rst:28-33
+To build a fuzzer, run the desired ninja command for the fuzzer(s) you want to
build:
+
+::
+ $ ninja lldb-target-fuzzer
+ $ ninja lldb-commandinterpreter-fuzzer
+ $ ninja lldb-expressio
JDevlieghere added inline comments.
Comment at: lldb/docs/resources/fuzzing.rst:14-26
+::
+ $ cmake \
+-G Ninja \
+-DCMAKE_BUILD_TYPE='Release' \
+-DLLVM_USE_SANITIZER='Address' \
+-DLLVM_USE_SANITIZE_COVERAGE=On \
+-DLLVM_BUILD_RUNTIME=
mib added a comment.
Left few comments :)
Comment at: lldb/docs/resources/fuzzing.rst:12
+
+Building the LLDB fuzzers requires a build configuration that has the address
sanitizer and sanitizer coverage. This CMake invocation will configure a build
directory that can be used
cassanova created this revision.
cassanova added reviewers: JDevlieghere, mib.
cassanova added a project: LLDB.
Herald added a subscriber: arphaman.
Herald added a project: All.
cassanova requested review of this revision.
Herald added a subscriber: lldb-commits.
This commit adds a new page to the
