[libcxx-commits] [PATCH] D154140: [libc++] Add .fuzz.cpp tests and move the fuzzing tests to the normal locations

[Mark de Wever via Phabricator via libcxx-commits]

Mordante added a comment.

In D154140#4485596 <https://reviews.llvm.org/D154140#4485596>, @philnik wrote:

> In D154140#4485554 <https://reviews.llvm.org/D154140#4485554>, @Mordante wrote:
>
>> Thanks for working on this! Should we mention fuzzing on the libc++ contributing page. Until this patch I wasn't even aware we had a fuzzing framework. I would say that format should have fuzzing too, @philnik already asked about it, and I will do it.
>
> I think we should first find out what we actually want to fuzz and how to properly do it before we mention anything on the contributing page. After all, these should be just additional coverage, not the main thing we use for testing.

Fair point. My general idea is to fuzz everything where the library acts on user defined input. I guess that is mainly parsers, where I consider format specifiers and regexes also parsing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154140