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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits