[PATCH] D119000: [Bitcode] Add fuzzer for bitcode reading

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 4 12:28:30 PST 2022


dblaikie accepted this revision.
dblaikie added a comment.
This revision is now accepted and ready to land.

In D119000#3297608 <https://reviews.llvm.org/D119000#3297608>, @nikic wrote:

> In D119000#3297487 <https://reviews.llvm.org/D119000#3297487>, @dblaikie wrote:
>
>> If we check this in, does it start failing buildbots or anything if it's not clean? Do we have to address the OOMs for this to be clean, or are OOMs not considered failures by any buildbots/don't produce failmail?
>
> Nothing should use the fuzzer by default, it needs to be added to oss-fuzz to run. So just adding it is not problematic.

Hmm, seems unfortunate to checkin dead code, though? But I guess if this is the normal way fuzzers are added/adopted/eventually integrated into oss-fuzz, fair enough.


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

https://reviews.llvm.org/D119000



More information about the llvm-commits mailing list