diff options
| author | jackkoenig | 2015-11-24 19:18:01 -0800 |
|---|---|---|
| committer | jackkoenig | 2015-11-24 19:18:01 -0800 |
| commit | 71f0319e8d27d1f175b4747c0367843a6ceab986 (patch) | |
| tree | acd1c96a30f4b04fcc6202cd9f9f59c9e9de432a /src/main/antlr4/FIRRTL.g4 | |
| parent | 876c5e0a93c21a2108804eb972289f77c250e28b (diff) | |
In process of adding FAME-1 transformation, updated todos in grammar file, updated Makefile to play nicer when firrtl is a submodule, fixed bug in Translator for single line scopes, fixed firrtl-scala script to point to firrtl.Driver instead of old firrtl.Test
Diffstat (limited to 'src/main/antlr4/FIRRTL.g4')
| -rw-r--r-- | src/main/antlr4/FIRRTL.g4 | 20 |
1 files changed, 8 insertions, 12 deletions
diff --git a/src/main/antlr4/FIRRTL.g4 b/src/main/antlr4/FIRRTL.g4 index 96172895..dd270d17 100644 --- a/src/main/antlr4/FIRRTL.g4 +++ b/src/main/antlr4/FIRRTL.g4 @@ -4,17 +4,20 @@ grammar FIRRTL; * PARSER RULES *------------------------------------------------------------------*/ -// TODO add [info] support (all over the place) -// TODO Fix connect -// TODO Add partial connect -// TODO Should FIRRTL keywords be legal IDs? +/* TODO + * - Add [info] support (all over the place) + * - Add support for indexers + * - Add support for extmodule + * - Fix connect + * - Add partial connect + * - Should FIRRTL keywords be legal IDs? +*/ // Does there have to be at least one module? circuit : 'circuit' id ':' '{' module* '}' ; -// TODO Add support for extmodule module : 'module' id ':' '{' port* blockStmt '}' ; @@ -40,11 +43,6 @@ field : orientation id ':' type ; -// FIXME This is what the spec says it should be -//orientation -// : 'default' -// | 'reverse' -// ; orientation : 'flip' | // Nothing @@ -166,8 +164,6 @@ IdNondigit | [~!@#$%^*-+=?/] ; -// Should enforcing signed, non-neg, and positive ints be done in parser? -// => YES IntLit : '0' | ( '+' | '-' )? [1-9] ( Digit )* |