[PATCH] D129093: [pseudo] Eliminate LRTable::Action. NFC

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 5 03:27:46 PDT 2022


hokein accepted this revision.
hokein added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang-tools-extra/pseudo/include/clang-pseudo/grammar/LRTable.h:84
-
-      // NOTE: there are no typical accept actions in the LRtable, accept
-      // actions are handled specifically in the parser -- if the parser
----------------
I think this comment has some value, it is worth to keep it in some form (might be append to the LRTable class comment).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129093



More information about the cfe-commits mailing list