[PATCH] D55083: Re-arrange content in FunctionDecl dump

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added a comment.

The summary explains what it's doing, but not why it's doing it -- why is this change in behavior needed? Does this not break any tests?

Btw, as we work on this refactoring, I think a good approach will be to build up the base of tests around AST dumping so that we can be explicitly aware of any behavioral changes from the patches. We have some coverage, but it doesn't look to be particularly comprehensive. I am happy to contribute some of these tests. WDYT?


Repository:
  rC Clang

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

https://reviews.llvm.org/D55083