[PATCH] D80712: [SVE] Add checks for no warnings in SVE tests

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 18 08:07:16 PDT 2020


thakis added inline comments.


================
Comment at: clang/test/CodeGen/aarch64-sve-intrinsics/acle_sve_adda.c:3
 // RUN: %clang_cc1 -D__ARM_FEATURE_SVE -DSVE_OVERLOADED_FORMS -triple aarch64-none-linux-gnu -target-feature +sve -fallow-half-arguments-and-returns -S -O1 -Werror -Wall -emit-llvm -o - %s | FileCheck %s
+// RUN: %clang_cc1 -D__ARM_FEATURE_SVE -triple aarch64-none-linux-gnu -target-feature +sve -fallow-half-arguments-and-returns -S -O1 -Werror -Wall - %s >/dev/null 2>%t
+// RUN: FileCheck --check-prefix=ASM --allow-empty %s <%t
----------------
This needs a `-o -`, or better a `-o /dev/null` instead of the piping to /dev/null. As-is, this creates a .s file in the test dir next to this file which on the next run is executed as a test that's unresolved. (Add a `rm -f %S/acle_sve_adda.s` to clean up bots, with a FIXME to remove this after a few days / weeks.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80712/new/

https://reviews.llvm.org/D80712





More information about the llvm-commits mailing list