[PATCH] [AArch64] Implement Clang CLI interface proposal about "-march".

Kristof Beyls kristof.beyls at arm.com
Wed Jul 16 00:32:34 PDT 2014


I think there's one more missing test or set of tests, to cover this sentence from the documentation of the -mcpu option:
"Where this option is used in conjunction with -march or -mtune, those options take precedence over the appropriate part of this option."

Apart from that, this patch looks good to me.

================
Comment at: lib/Driver/Tools.cpp:1568
@@ +1567,3 @@
+
+// Handle -march=.
+static void getAArch64FeaturesFromMtune(const Driver &D, StringRef Mtune,
----------------
-march -> -mtune in the comment.
TBH, I don't think these comments add much value, the name of the functions are descriptive enough.

http://reviews.llvm.org/D4346






More information about the llvm-commits mailing list