hokein added a comment. New data on the patch:
| file | ambiguous nodes | forest size | glrParse performance | | SemaCodeComplete.cpp | 11k -> 5.7K | 10.4MB -> 7.9MB | 7.1MB/s -> 9.98MB/s | | AST.cpp | 1.3k -> 0.73K | 0.99MB -> 0.77MB | 6.7MB/s -> 8.4MB/s | | ================ Comment at: clang-tools-extra/pseudo/lib/cxx/cxx.bnf:39 namespace-name := IDENTIFIER -namespace-name := namespace-alias -namespace-alias := IDENTIFIER class-name := IDENTIFIER class-name := simple-template-id ---------------- this can be removed as well, but the `class-head-name` rule uses it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130747/new/ https://reviews.llvm.org/D130747 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
