[PATCH] D91573: [YAMLIO] Add a generic YAML fuzzer harness

Scott Linder via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 15:08:24 PST 2020


scott.linder added a comment.

In D91573#2403817 <https://reviews.llvm.org/D91573#2403817>, @dexonsmith wrote:

> I suggest changing this patch to include the `isValidYaml` function from https://reviews.llvm.org/D84050 and call that, and then that revision can be rebased on top of this one. If you do that, this LGTM.

This is what I ended up doing, thank you! I also adopted the `vector<uint8_t>` style here and copied back the matching comment from the next patch. Let me know if you notice any mistakes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91573



More information about the llvm-commits mailing list