aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
LGTM aside from the testing situation -- so long as the tests come out without
changes, this is good to go. If there is a change in test behavior, let's do
another round of review just to verify everyone's happy with the behavioral
change.
================
Comment at: lib/AST/ASTDumper.cpp:1232
- if (D->isThisDeclarationADefinition()) {
+ if (D->isThisDeclarationADefinition())
dumpDeclContext(D);
----------------
Since this is just formatting changes unrelated to the patch, feel free to
revert and commit separately for commit hygiene.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56829/new/
https://reviews.llvm.org/D56829
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
