[PATCH] D101387: [Clang] remove text extension from diag::err_drv_invalid_value_with_suggestion
Nick Desaulniers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon May 3 11:18:16 PDT 2021
nickdesaulniers 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`
Can they? If they did, then flang would hard code "'none' or a positive integer" and " or greater" which I think defeats the purpose of what @rsmith asked for when referring to https://clang.llvm.org/docs/InternalsManual.html#the-format-string.
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