[PATCH] D85330: [SyntaxTree] Extend the syntax tree dump

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 5 12:34:04 PDT 2020


gribozavr2 added inline comments.


================
Comment at: clang/lib/Tooling/Syntax/Tree.cpp:158
   if (!Marks.empty())
     OS << Marks << ": ";
 
----------------
Maybe the marks should be moved after the primary identifier of the node, WDYT? That would be more consistent with AST dump: first the node kind, then all its info.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85330



More information about the cfe-commits mailing list