[flang-commits] [flang] [flang] Add REQUIRES for the trampoline test (PR #185699)
via flang-commits
flang-commits at lists.llvm.org
Tue Mar 10 10:25:43 PDT 2026
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-driver
Author: Kelvin Li (kkwli)
<details>
<summary>Changes</summary>
Instead of listing the UNSUPPORTED list, I think it makes sense to have the REQUIRES. The REQUIRES list matches https://github.com/llvm/llvm-project/blob/1fdc16bd426f3db2aa772cc3a8dff33ec8e91e76/clang/lib/Driver/ToolChains/Flang.cpp#L209-L211
---
Full diff: https://github.com/llvm/llvm-project/pull/185699.diff
1 Files Affected:
- (modified) flang/test/Driver/fsafe-trampoline.f90 (+1-1)
``````````diff
diff --git a/flang/test/Driver/fsafe-trampoline.f90 b/flang/test/Driver/fsafe-trampoline.f90
index 7318e7ea6f606..bfe124e447cf7 100644
--- a/flang/test/Driver/fsafe-trampoline.f90
+++ b/flang/test/Driver/fsafe-trampoline.f90
@@ -1,7 +1,7 @@
! Test that -fsafe-trampoline is properly forwarded from driver to
! frontend, and that -fno-safe-trampoline (default) works.
-! UNSUPPORTED: system-aix
+! REQUIRES: target=aarch64{{.*}} || target=x86{{.*}}
! RUN: %flang -### -fsafe-trampoline %s 2>&1 | FileCheck %s --check-prefix=ON
! RUN: %flang -### -fno-safe-trampoline %s 2>&1 | FileCheck %s --check-prefix=OFF
``````````
</details>
https://github.com/llvm/llvm-project/pull/185699
More information about the flang-commits
mailing list