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

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 6 00:22:25 PDT 2022


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 another name -- `O` looks similar to the number literal `0`.  
Realized I can use std::remove_if here instead which is cleaner.


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