[PATCH] D125006: [pseudo] Support parsing variant target symbols.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 11 05:32:38 PDT 2022
hokein added a comment.
> Would including *every* nonterminal as a start symbol would blow the size up a bit?
> This would eliminate some complexity in the interface.
This does increase the size, some datapoints:
- ~240 start symbols
- number of states: 1.4K -> 2K
- number of actions: 83K -> 96K
- size of table(bytes): 334K -> 388K
As discussed, the benefit is not obvious, let's leave it out now and revisit it when needed (added a comment in the cxx.bnf).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125006/new/
https://reviews.llvm.org/D125006
More information about the cfe-commits
mailing list