[flang-commits] [flang] [flang][test] Run Driver/fveclib-codegen.f90 for x86_64 (PR #103730)

[Rainer Orth via flang-commits]

================
@@ -1,7 +1,8 @@
 ! test that -fveclib= is passed to the backend
 ! -target aarch64 so that ArmPL is available
-! RUN: %flang -S -Ofast -fveclib=LIBMVEC -o - %s | FileCheck %s
-! RUN: %flang -S -Ofast -fveclib=NoLibrary -o - %s | FileCheck %s --check-prefix=NOLIB
+! REQUIRES: x86-registered-target
----------------
rorth wrote:

> I don't fully understand what the issue is - the tests are running on AArch64 today already, right?

Sure, but the test is always using the native target, irrespective if that supports `libmvec` (`aarch64` or `x86_64` only) or not.  Everywhere else, the test is guaranteed to `FAIL` as is.

> Also, this LIT example might be helpful:
> 
> https://github.com/llvm/llvm-project/blob/2c8bd4a729a042ed823547ea0eacad429c50d7bf/flang/test/Driver/tune-cpu-fir.f90#L5

I'd meanwhile found `flang/test/Lower/tune-cpu-llvm.f90` which is simliar.  This is the first time I learned about this`'%if` feature **without** the need for `REQUIRES: *-registered-target`.

https://github.com/llvm/llvm-project/pull/103730