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



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

Reply via email to