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

Lev Aronsky via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 27 05:01:10 PDT 2020


aronsky added a comment.

In D90221#2356062 <https://reviews.llvm.org/D90221#2356062>, @lebedev.ri wrote:

> Are there tests missing?

Quite possible. I followed the trail of the existing functions to figure out the difference between JSON and textual dumping, and tried replicating everything in a manner similar to the existing code. I haven't run into any tests, but that's probably because I wasn't looking for those. I'll add the appropriate tests ASAP.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90221/new/

https://reviews.llvm.org/D90221



More information about the cfe-commits mailing list