[PATCH] D130551: [pseudo] Allow opaque nodes to represent terminals

2022-07-26 Thread Sam McCall via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG07b7ff983837: [pseudo] Allow opaque nodes to represent terminals (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D130551: [pseudo] Allow opaque nodes to represent terminals

2022-07-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. In D130551#3679021 , @hokein wrote: > The change looks good, are you planning to use it in the cxx.bnf? I want to land a recovery rule that completes scopes (e.g. namespaces) rather than discarding them. But i have to fix sev

[PATCH] D130551: [pseudo] Allow opaque nodes to represent terminals

2022-07-26 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. The change looks good, are you planning to use it in the cxx.bnf? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130551/new/ https://reviews.llvm

[PATCH] D130551: [pseudo] Allow opaque nodes to represent terminals

2022-07-26 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added reviewers: hokein, usaxena95. Herald added a project: All. sammccall requested review of this revision. Herald added subscribers: cfe-commits, alextsao1999. Herald added a project: clang-tools-extra. This allows incomplete code such as `namespace fo