DavidSpickett added inline comments.

================
Comment at: clang/lib/Driver/ToolChains/Arch/ARM.cpp:476
+    // -mfpu=none, -march=armvX+nofp or -mcpu=X+nofp is *very* similar to
+    // -mfloat-abi=soft, only that it should not disable MVE-I.
     Features.insert(Features.end(),
----------------
Why not disable MVE-I? I assume because it's integer only but then why does 
-mfloat-abi=soft disable it?

If possible add a regression test for this. In general a test like the bf16 
test below, but for all the listed extensions would help. Perhaps it makes more 
sense to add a driver test that looks for the "-<ext>" bits in the -### output 
instead of doing each extension on its own.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82948



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

Reply via email to