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

Tue Jul 26 03:37:39 PDT 2022

sammccall added a comment.

In D130551#3679021 <https://reviews.llvm.org/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 several deficiencies, at least this one, D130550 <https://reviews.llvm.org/D130550>, D130523 <https://reviews.llvm.org/D130523>. and at some point we have to allow rules to have more than one recovery point.

I like having the motivating case in the patch itself but in this case there's just too much of this stuff

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130551