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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to