[PATCH] D135312: [libFuzzer] update the libFuzzer docs to reflect the current state.

Aaron Ballman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 10 12:25:30 PDT 2022


aaron.ballman added a comment.

In D135312#3847599 <https://reviews.llvm.org/D135312#3847599>, @kcc wrote:

>> I spotted a typo
>
> Thanks!
>
>> Was there an RFC for this change? I'm very surprised to read "However please do not expect ... code reviews, other than for bug fixes." for an open source project that's maintained by the community. Either this is a supported feature and the community should review whatever patches we get for it on their own merits, or we should officially end-of-life this functionality and talk about timelines for removal.
>
> There was no RFC. This change simply documents the state as of 2-3 years now. 
> libFuzzer is in wide use currently, I don't think we should EOL it for some time now. 
> Once Centipede is at feature parity with libFuzzer I will propose an EOL.

On the one hand, documenting reality is always a good thing. On the other hand, I think this needs community buy-in around what message we want to send users. I've already started seeing off-list complaints about suddenly dropping support for a proven technology in favor of an unproven one, and I share that concern. This isn't to suggest the authors of Centipede shouldn't be working on that project! It's more to point out that saying libFuzzer is supported but not actively worked on may not be the messaging we want to send users (as several folks seem to read this as "not supported" despite the docs saying "fully supported", and there may be community members who want to step up to actively support the project to add new features, minor bug fixes, etc.

>> Also, can the code owners file for LLVM please be updated with the name(s) of whoever is doing the limited maintenance activities?
>
> compiler-rt/CODE_OWNERS.TXT needs lots of changes other than that. 
> I'll ask folks to do it.

Thank you! <3


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135312/new/

https://reviews.llvm.org/D135312



More information about the llvm-commits mailing list