- User Since
- Sep 5 2013, 6:50 AM (311 w, 1 d)
May 19 2019
May 17 2019
I'm going to investigate whether we can move the dump method implementations to their respective class files, and then look into a rename for this to StreamNodeDumper or so (name tbd).
May 16 2019
Add basic traverser test
Thanks, I made the comment changes in a separate commit because otherwise this would cease to be a refactoring commit. Commits get too noisy if their content is modified "mid-flight".
May 13 2019
Indeed, the cmake_minimum_required command sets policies to new.
May 12 2019
@klimek This includes a test for the memoization case you were interested in at EuroLLVM.
This is part of the work I demo'd at EuroLLVM for ignoring invisible AST nodes during AST Matching and dumping: http://ce.steveire.com/z/lHYwEH
Apr 29 2019
Thanks for doing this! I'm glad you were able to do it without needing to change the traverser class. That's a good indicator.
Feb 5 2019
Sorry, this fell off my radar :). LGTM too.
Feb 3 2019
I have no objection to this, but I wonder whether all state accessible from all nodes should be part of the AST dump. Where do you think the line is? Is there anything else missing currently from other nodes?
Feb 1 2019
Jan 31 2019
Jan 30 2019
Jan 29 2019
An alternative was written and committed.
This was subsequently reverted. Is there a status update? Rebasing and committing D56959 would unblock traverser work and would allow this to progress separately in parallel.
Jan 26 2019
I was just working on this this afternoon to show you the difference when it is split up. Looks like you committed this meanwhile.
Jan 25 2019
Jan 24 2019
There's definitely a better possible ordering in two commits: