[PATCH] D96347: Include function return type in JSON AST dumps.

2021-03-02 Thread Rokas Kupstys via Phabricator via cfe-commits
rokups added a comment. Hey @Temtaime, Until we can rely on this i opted in for poor man's workaround - splitting type/qualType field at the first ( and using first part. I realize it is not reliable and will break when function pointers are used, but since they arent in my project that is an a

[PATCH] D96347: Include function return type in JSON AST dumps.

2021-02-11 Thread Rokas Kupstys via Phabricator via cfe-commits
rokups added a comment. This seems to not be good enough, however. Somehow desugared version of type is not added anywhere. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96347/new/ https://reviews.llvm.org/D96347 _

[PATCH] D96347: Include function return type in JSON AST dumps.

2021-02-09 Thread Rokas Kupstys via Phabricator via cfe-commits
rokups created this revision. rokups requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. For some reason return type was not included. Even though it is present in full type dump, it is not good enough as C++ types can get quite complex. Beside