[PATCH] D36731: [ARM][AArch64] Cortex-A75 and Cortex-A55 tests

Sjoerd Meijer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 18 07:09:30 PDT 2017

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

Looks good to me too.

Two nits (no new review required): one is inlined, and the other one in the summary: ARMv8.2-A => Armv8.2-A   :-/

Comment at: test/Driver/arm-dotprod.c:2
+// RUN: %clang -### -target arm %s 2>&1 | FileCheck %s --check-prefix=CHECK-NONE
+// RUN: %clang -### -target arm -march=armv8.1a %s 2>&1 | FileCheck %s --check-prefix=CHECK-NONE
+// RUN: %clang -### -target arm -march=armv8.2a %s 2>&1 | FileCheck %s --check-prefix=CHECK-NONE
Perhaps also test "v8.0" and add -march=armv8a?


More information about the cfe-commits mailing list