[PATCH] D85316: [SyntaxTree] Proposition of new tree dump

Dmitri Gribenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 5 09:10:44 PDT 2020


gribozavr2 added a comment.

I'd suggest to drop the `<>` quotes (because the AST dump does not add quotes unless it is printing a multi-word thing, and because `<>` don't exactly scream "role" helping to read the output).

I'd also suggest to replace multiple spaces before `<>` with a single space.

Otherwise, looks great!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85316



More information about the cfe-commits mailing list