[PATCH] D156237: Complete the implementation of P2361 Unevaluated string literals

Corentin Jabot via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 27 06:57:05 PDT 2023


cor3ntin marked 2 inline comments as done.
cor3ntin added inline comments.


================
Comment at: clang/include/clang/Basic/DiagnosticCommonKinds.td:60
+          "'external_source_symbol' attribute|"
+          "as argument of '%1' attribute}0">;
+
----------------
aaron.ballman wrote:
> This duplicates part of `err_attribute_argument_type`; probably worth it to update that diagnostic so we get consistent diagnostic wording between attributes expecting string literal arguments.
I'm not sure there is a good way to do that without changing thousands of tests. I did look into it!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156237



More information about the cfe-commits mailing list