[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