aaron.ballman accepted this revision.
aaron.ballman added a comment.
LGTM with a small nit. @ABataev, are you okay with this approach?
================
Comment at: lib/AST/ASTDumper.cpp:1060
+ dumpStmt(D->getCombiner());
+ if (auto *Initializer = D->getInitializer()) {
dumpStmt(Initializer);
----------------
Elide braces and may as well make this `const auto *`. (If you want to hit the
one above, that would also be good -- maybe even hoist the call into an
initializer for a local variable to be used in both places?)
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55395/new/
https://reviews.llvm.org/D55395
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
