xen0n marked 2 inline comments as done.
xen0n added inline comments.
================
Comment at: clang/include/clang/Basic/DiagnosticDriverKinds.td:732
+def warn_drv_loongarch_conflicting_mabi : Warning<
+  "the -mabi setting '%0' conflicts with that implied by -m*-float (%1); using 
%1">,
+  InGroup<OptionIgnored>;
----------------
MaskRay wrote:
> This is not called `setting`. Use `value`
> 
> It seems that `ignoring -mabi value '%0' as it conflicts ...` is more 
> conventional.
Sounds reasonable. I've checked other similar diagnostics text and it seems 
your suggestion is stylistically more consistent.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153707

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to