[PATCH] D101387: [Clang] remove text extension from diag::err_drv_invalid_value_with_suggestion

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 3 11:17:46 PDT 2021


xbolva00 added inline comments.


================
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:279
   "-fembed-bitcode is not supported on versions of iOS prior to 6.0">;
+def err_drv_negative_columns : Error<
+    "invalid value '%1' in '%0', value must be 'none' or a positive integer">;
----------------
MaskRay wrote:
> The two can reuse `err_drv_invalid_value_with_suggestion`
We dont need to reuse everything at all cost. Clear and understandable messages are far more important.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101387



More information about the cfe-commits mailing list