[flang-commits] [flang] [flang][test] Fix REQUIRES and options for sum.f90 (PR #146872)

[via flang-commits]

llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

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

Author: David Spickett (DavidSpickett)

<details>
<summary>Changes</summary>

This should have been looking for the "x86" target not "x64_64".

When run on AArch64 it failed because bbc tried to compile for AArch64. Add a target option to fix that, as this test is x86 specific.

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


1 Files Affected:

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


``````````diff
diff --git a/flang/test/Lower/Intrinsics/sum.f90 b/flang/test/Lower/Intrinsics/sum.f90
index d36b2b1263887..3167617b60457 100644
--- a/flang/test/Lower/Intrinsics/sum.f90
+++ b/flang/test/Lower/Intrinsics/sum.f90
@@ -1,5 +1,5 @@
-! REQUIRES: x86_64-registered-target
-! RUN: bbc --use-desc-for-alloc=false -emit-fir -hlfir=false %s -o - | FileCheck %s
+! REQUIRES: x86-registered-target
+! RUN: bbc -target x86_64-unknown-linux-gnu --use-desc-for-alloc=false -emit-fir -hlfir=false %s -o - | FileCheck %s
 
 ! CHECK-LABEL: func @_QPsum_test(
 ! CHECK-SAME: %[[arg0:.*]]: !fir.box<!fir.array<?xi32>>{{.*}}) -> i32 {

``````````

</details>


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