[PATCH] D61834: Add a Visit overload for DynTypedNode to ASTNodeTraverser

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 17 07:02:57 PDT 2019


aaron.ballman added inline comments.


================
Comment at: unittests/AST/ASTTraverserTest.cpp:75
+
+template <typename... NodeType> std::string dumpASTString(NodeType &&... N) {
+  std::string Buffer;
----------------
steveire wrote:
> aaron.ballman wrote:
> > Did clang-format produce this formatting? (If not, run through clang-format.)
> Yep, clang-format made it like this.
Weird that it added the extra whitespace -- I think this is the only instance where we have whitespace to either side of the adornments, which is what caught me by surprise.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61834





More information about the cfe-commits mailing list