[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