[PATCH] Allow case-insensitive values for -mcpu for AArch64 target in line with GCC.
Richard Barton
richard.barton at arm.com
Fri Jun 12 09:22:04 PDT 2015
I have one comment about a missing regression test, but otherwise this patch seems fine to me. I understand the previous version broke the ARM bot, so I will let Renato have last say.
REPOSITORY
rL LLVM
================
Comment at: test/Driver/aarch64-cpus.c:114
@@ -113,1 +113,3 @@
+// ================== Check whether -mcpu accepts mixed-case values.
+// RUN: %clang -target aarch64 -mcpu=Cortex-a53 -### -c %s 2>&1 | FileCheck -check-prefix=CASE-INSENSITIVE-CA53 %s
----------------
It seems your patch also allows mixed case in the +feature part of the argument. This is in line with GCC 4.9, but I would expect a regression test checking that here.
http://reviews.llvm.org/D10412
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the cfe-commits
mailing list