Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Promote the rrd node tree to serve as a general IR for all dialects #24

Open
katef opened this issue Mar 11, 2019 · 0 comments
Open

Promote the rrd node tree to serve as a general IR for all dialects #24

katef opened this issue Mar 11, 2019 · 0 comments

Comments

@katef
Copy link
Owner

katef commented Mar 11, 2019

The AST is inconvenient; it has nesting of a forced hierarchy of particular types. In retrospect I think I'd prefer to construct rrd nodes directly. And now the tnode tree has positional information split off, I think these rrd nodes should serve well as an abstract representation of grammars for both transformation and for rendering to all outputs. In turn that makes the AST nodes redundant, and I think those can go.

So:

  1. Move the rrd nodes out for general use;
  2. Switch all outputs to the rrd nodes;
  3. Have all outputs share the tree rewriting;
  4. Construct the rrd nodes directly from parser actions;
  5. Remove AST;

And perhaps:
6. Rename rrd nodes to AST
7. Rename tnode to rrd?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant