- User Since
- Sep 5 2013, 6:50 AM (327 w, 20 h)
Tue, Dec 10
It wasn't possible to add the const because of the return type.
Sat, Dec 7
Fri, Dec 6
Committed in 2e8dc8590d8b412
Committed in b22d8ae7f436bfe63. I don't know why this review didn't automatically update.
I tried using DenseMap, but couldn't make it compile. I stuck with std::map for now.
Fri, Nov 22
Rebase and update
Oct 14 2019
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.