[PATCH] D76801: [AST] Print a<b<c>> without extra spaces in C++11 or later.

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 25 14:38:47 PDT 2020


sammccall added a comment.

@kadircet sending this to you as I noticed it as a clangd hover bug :-) The highest impact is probably in diagnostic messages though.

AFAICT the state before this patch is:

- the type printer always printed the space
- the stmt printer calls into the type printer to print arg lists
- the decl printer never prints the space

The best, most consistent thing would be to also fix the decl printer to respect the setting. But I haven't done that in this patch. At least the default configuration (C++14) is now consistent between type/stmt/decl.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76801





More information about the cfe-commits mailing list