[PATCH] D82948: [Driver][ARM] Disable unsupported features when nofp arch extension is used

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 29 04:04:56 PDT 2020


chill accepted this revision.
chill added a comment.
This revision is now accepted and ready to land.

LGTM, thanks!



================
Comment at: clang/lib/Driver/ToolChains/Arch/ARM.cpp:481
+                    {"-dotprod", "-fp16fml", "-bf16", "-mve.fp"});
+    if (!hasIntegerMVE(Features)) {
       Features.emplace_back("-fpregs");
----------------
LLVM coding standards call for not using braces on single-statement bodies and that's also the style in this source file.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82948



More information about the llvm-commits mailing list