[PATCH] D125231: [pseudo] Compile cxx grammar.

2022-05-09 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. Cool! High level thoughts: **Motivation** What's the win here? performance of loading/compiling the grammar? self-contained-ness? how much are we saving? Doing the compilation at build time seems likely to be good for performance, but we should have numbers. And there

[PATCH] D125231: [pseudo] Compile cxx grammar.

2022-05-09 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. This is just a prototype, wanting some early feedback before making further progress: - compiling the generated Cxx.cpp is very slow (took minutes, mostly due to the LRTable::Actions); - layering (location of the generated header file) is not super clear; Repository:

[PATCH] D125231: [pseudo] Compile cxx grammar.

2022-05-09 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Herald added a subscriber: mgorny. Herald added a project: All. hokein requested review of this revision. Herald added a subscriber: alextsao1999. Herald added a project: clang-tools-extra. It compiles the cxx bnf grammar, and gener