Robin
Robin
I was struck by this issue again today. The workaround is in commit https://github.com/llir/grammar/commit/1b802f1297e7e7938df92025967d6fc679da5df2 In essence I have an `OffsetField` nonterminal with the `Offset` attribute in the LLVM IR grammar....
It looks as though this issue has been resolved. Given the grammar [a.tm](https://github.com/mewspring/foo/blob/master/issue3/a.tm), the output is [ast/Ast_Node.java](https://github.com/mewspring/foo/blob/master/issue3/ast/Ast_Node.java#L5): ``` public class Ast_Node extends AstNode { ``` As such. Feel free to...
Below follows a minimal reproducible example which exhibits this issue [issue25.tm](https://github.com/mewspring/foo/blob/master/issue25/issue25.tm): ```c language issue25(go); lang = "issue25" package = "github.com/mewspring/foo/issue25" eventBased = true eventFields = true eventAST = true ::...
For the specific use case above, I found the following workaround: Essentially, make `UnwindTarget` a concrete type instead of a sum type (i.e. interface). ``` UnwindTarget -> UnwindTarget : UnwindToCaller...
Thanks for the write-up @inspirer! It will take me some time to digest. > All children nodes within one node a distinguished by their type only, which simplifies (read speeds...
We now switched to using the workaround you proposed in https://github.com/inspirer/textmapper/issues/25#issuecomment-437503739 and it works well (implemented in rev https://github.com/llir/grammar/commit/b01e345b685dd61b51787d1255a9ddfd0b6c9baf). This allowed us to once more have `UnwindTarget` be defined as...
> To avoid (forbid?) nullable nonterminals at first and last positions in a rule. This is the rule I'm currently following everywhere. Textmapper should flag when a nullable terminal is...
I tried to manually inline the instruction metadata attachments in the LLVM IR grammar. It works. I get `unknown node type NONE` before inline, and successful resolution after inlining (i.e....
Just to add to the discussion, if approach 2 tracks the parent of each non-terminal, perhaps such a solution could also allow for non-terminals to appear at different locations within...
For reference to others coming across this issue, @inspirer wrote a detailed response to this issue in PR #35, which is included below: > Sorry for not getting to this...