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

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 5 22:25:38 PDT 2022


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: I'd suggest using another name -- `O` looks similar to the number literal `0`.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125037



More information about the cfe-commits mailing list