[PATCH] D60699: [ARM] add CLI support for 8.1-M and MVE.
Sjoerd Meijer via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 15 06:09:45 PDT 2019
SjoerdMeijer added inline comments.
================
Comment at: clang/test/Driver/armv8.1m.main.c:1
+// RUN: %clang -target arm-arm-none-eabi -march=armv8.1-m.main+dsp -### %s 2> %t
+// RUN: FileCheck --check-prefix=CHECK-DSP < %t %s
----------------
It doesn't really matter, I guess, but we don't need a temp file and can pipe the output directly to FileCheck?
================
Comment at: clang/test/Driver/armv8.1m.main.c:3
+// RUN: FileCheck --check-prefix=CHECK-DSP < %t %s
+// CHECK-DSP: "-target-feature" "+dsp"
+
----------------
Do we also want to check that just:
-march=armv8.1-m
doesn't enable DSP (and other non-mandatory extensions)?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60699/new/
https://reviews.llvm.org/D60699
More information about the cfe-commits
mailing list