This adds a new extern "C" function that serves the same purpose. This removes the need for external users to depend on internal headers in order to use this feature. It also standardizes the interface in a way that other fuzzing engines will be able to match.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Time | Test | |
---|---|---|
140 ms | linux > Polly.ScopInfo::Unknown Unit Message ("") |
Event Timeline
Comment Actions
Revert of the original changes was already done in 540b92147da0e9a2c2d096505eb0785ec79f2024 :) - a rebase should fix it up.
I'll let @kcc take this one.
Comment Actions
Code LGTM, thanks!
Please add a section in docs/LibFuzzer.html.
I'd add it after "Startup initialization", something like "Using libFuzzer as a library".
I assume you will also need to introduce a .so build target.
Will you do it in a separate change?
clang-tidy: warning: invalid case style for function 'LLVMFuzzerRunDriver' [readability-identifier-naming]
not useful
clang-tidy: warning: invalid case style for parameter 'argc' [readability-identifier-naming]
not useful
clang-tidy: warning: invalid case style for parameter 'argv' [readability-identifier-naming]
not useful