Page MenuHomePhabricator

[FuzzMutate] Split out FuzzerCLI library that doesn't depend on IR.
ClosedPublic

Authored by sammccall on May 6 2022, 1:37 AM.

Details

Summary

All llvm-project fuzzers use this library to parse command-line arguments.
Many of them don't deal with LLVM IR or modules in any way. Bundling those
functions in one library forces build dependencies that don't need to be there.

Among other things, this means check-clang-pseudo no longer depends on most of
LLVM.

Diff Detail

Event Timeline

sammccall created this revision.May 6 2022, 1:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 6 2022, 1:37 AM
sammccall requested review of this revision.May 6 2022, 1:37 AM
Herald added projects: Restricted Project, Restricted Project, Restricted Project, Restricted Project. · View Herald TranscriptMay 6 2022, 1:37 AM

I'm happy to move FuzzerCLI to a separate directory (and avoid PARTIAL_SOURCES_INTENDED) if you prefer.

It's a bunch more mechanical changes and will break secondary build systems (gn, bazel), so if so let's agree on a name first.

hokein accepted this revision.May 6 2022, 2:39 AM

This looks a reasonable split to me!

This revision is now accepted and ready to land.May 6 2022, 2:39 AM
This revision was landed with ongoing or failed builds.May 7 2022, 3:13 AM
This revision was automatically updated to reflect the committed changes.

Thanks for reverting, looks like I missed a target, sorry about that.