================
@@ -127,6 +127,11 @@ void loongarch::getLoongArchTargetFeatures(const Driver &D,
                                            const llvm::Triple &Triple,
                                            const ArgList &Args,
                                            std::vector<StringRef> &Features) {
+  // Enable the `lsx` feature on 64-bit LoongArch by default.
+  if (Triple.isLoongArch64() &&
+      (!Args.hasArg(clang::driver::options::OPT_march_EQ)))
----------------
MaskRay wrote:

Prefer `hasArgNoClaim` to have a single place to claim the option.
Unclaimed options will lead to an unused argument warning.

https://github.com/llvm/llvm-project/pull/100056
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to