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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
