[flang-commits] [flang] [flang] Fix failing test (PR #97634)

via flang-commits flang-commits at lists.llvm.org
Wed Jul 3 13:38:12 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-flang-fir-hlfir

Author: Mats Petersson (Leporacanthicus)

<details>
<summary>Changes</summary>

Add requires line to not test when the target architecture isn't supported.

Technically we could make it a bit less restrictive, but want green builds.

---
Full diff: https://github.com/llvm/llvm-project/pull/97634.diff


1 Files Affected:

- (modified) flang/test/Lower/mcmodel.f90 (+2) 


``````````diff
diff --git a/flang/test/Lower/mcmodel.f90 b/flang/test/Lower/mcmodel.f90
index dd9eb145f5e2a..971dc3b9056fe 100644
--- a/flang/test/Lower/mcmodel.f90
+++ b/flang/test/Lower/mcmodel.f90
@@ -1,3 +1,5 @@
+! REQUIRES:  aarch64-registered-target && x86-registered-target
+
 ! RUN: %flang_fc1 -triple aarch64 -emit-llvm -mcmodel=tiny %s -o - | FileCheck %s -check-prefix=CHECK-TINY
 ! RUN: %flang_fc1 -emit-llvm -mcmodel=small %s -o - | FileCheck %s -check-prefix=CHECK-SMALL
 ! RUN: %flang_fc1 -triple x86_64-unknown-linux-gnu -emit-llvm -mcmodel=kernel %s -o - | FileCheck %s -check-prefix=CHECK-KERNEL

``````````

</details>


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


More information about the flang-commits mailing list