[PATCH] D105759: Implement P2361 Unevaluated string literals

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 28 14:25:50 PDT 2023


hubert.reinterpretcast added a comment.

In D105759#4543246 <https://reviews.llvm.org/D105759#4543246>, @aaron.ballman wrote:

> I'd recommend we change the diagnostic to be a warning that defaults to an error so that users who are caught by the changes can still disable the diagnostic rather than be stuck; for Clang 18, we can explore other solutions to the issue. Would this work for you @hubert.reinterpretcast?

I think there are questions about whether an error (or even warning) by default is appropriate. This seems to be a change for C++2c that does not have "DR" treatment from the committee. Considering this a warning controlled by `c++2c-compat` is a potential direction. Indeed, if we are going to accept the code, we might as well allow it as an extension in C++2c modes. With this line of logic, I don't see why we would want user-side churn of making a migration effort.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105759



More information about the cfe-commits mailing list