[PATCH] D40476: Switch kryo to use -mcpu=cortex-a57 when invoking the assembler

Stephen Hines via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 27 09:24:43 PST 2017


srhines added inline comments.


================
Comment at: lib/Driver/ToolChains/Gnu.cpp:660
+    // of a cpu flag.
+    Arg *A = Args.getLastArg(options::OPT_mcpu_EQ);
+    if (A) {
----------------
Is it better to sink A into the if condition again?


================
Comment at: test/Driver/as-mcpu.c:1
+// ================== Check that krait is substituted by cortex-15 when invoking
+// the assembler
----------------
cortex-a15 - this is missing the "a".


================
Comment at: test/Driver/as-mcpu.c:6
+
+// ================== Check that kryo is substituted by cortex-57 when invoking
+// the assembler
----------------
cortex-a57


https://reviews.llvm.org/D40476





More information about the cfe-commits mailing list