[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