[PATCH] D104975: Implement P1949

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 26 06:42:01 PDT 2021


jfb added a comment.

It would be more user friendly to say which character is not allowed in the diagnostic.

Do we need to have a backwards compatible flag to preserve the old behavior, or do we believe that nobody will be affected by the change? We should make this choice explicitly and state it in the patch description.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104975



More information about the cfe-commits mailing list