[PATCH] D125037: [pseudo] Add fuzzer for the pseudoparser.

2022-05-06 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. This adds a false dependency between check-clang-pseudo and all of LLVM via the FuzzMutate library. (We use FuzzerCLI.h, but there's also a bunch of utilities for fuzzing IR). I'll try to split that library up to avoid the false dependency in a followup. Repository:

[PATCH] D125037: [pseudo] Add fuzzer for the pseudoparser.

2022-05-06 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. sammccall marked an inline comment as done. Closed by commit rG1616bd9ef4eb: [pseudo] Add fuzzer for the pseudoparser. (authored by sammccall). Changed prior to commit:

[PATCH] D125037: [pseudo] Add fuzzer for the pseudoparser.

2022-05-06 Thread Sam McCall via Phabricator via cfe-commits
sammccall marked an inline comment as done. sammccall added inline comments. Comment at: clang-tools-extra/pseudo/fuzzer/Fuzzer.cpp:81 + bool PrintForest = false; + unsigned O = 1; + for (int I = 1; I < *Argc; ++I) { hokein wrote: > nit: I'd suggest using

[PATCH] D125037: [pseudo] Add fuzzer for the pseudoparser.

2022-05-05 Thread Haojian Wu via Phabricator via cfe-commits
hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. Nice! Comment at: clang-tools-extra/pseudo/fuzzer/Fuzzer.cpp:81 + bool PrintForest = false; + unsigned O = 1; + for (int I = 1; I < *Argc; ++I) { nit:

[PATCH] D125037: [pseudo] Add fuzzer for the pseudoparser.

2022-05-05 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: hokein. Herald added a subscriber: mgorny. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, alextsao1999. Herald added a project: clang-tools-extra. As confirmation,