[PATCH] D90221: Include more attribute details when dumping AST in JSON

aaron.ballman added a subscriber: Tyker. aaron.ballman added a comment. In D90221#2381325 , @aronsky wrote: > In D90221#2379793 , @aaron.ballman > wrote: > >> I think the issue is that ASTNodeTraverser is visiting

[PATCH] D90221: Include more attribute details when dumping AST in JSON

aronsky added a comment. In D90221#2379793 , @aaron.ballman wrote: > I think the issue is that ASTNodeTraverser is visiting all of the attribute > arguments (around ASTNodeTraverse.h:726) and your patch also visits them (as > part of what's emitted from

[PATCH] D90221: Include more attribute details when dumping AST in JSON

aaron.ballman added a comment. In D90221#2374344 , @aronsky wrote: >> It seems surprising to me that you'd have to add those declarations; I think >> that's a layering violation. There's something somewhat strange going on >> here that I think is related

[PATCH] D90221: Include more attribute details when dumping AST in JSON

aronsky added a comment. > It seems surprising to me that you'd have to add those declarations; I think > that's a layering violation. There's something somewhat strange going on here > that I think is related. Given: > [...] > Notice how the annotate attribute has an `inner` array with the argu

[PATCH] D90221: Include more attribute details when dumping AST in JSON

aaron.ballman added a comment. In D90221#2358469 , @aronsky wrote: > In D90221#2357060 , @aaron.ballman > wrote: > >> In D90221#2356110 , @aronsky wrote: >> >>> In D90221#

[PATCH] D90221: Include more attribute details when dumping AST in JSON

aronsky updated this revision to Diff 302558. aronsky retitled this revision from "Include attribute details when dumping AST in JSON" to "Include more attribute details when dumping AST in JSON". aronsky edited the summary of this revision. aronsky added a comment. Updated the relevant unit-test