Skip to content

Commit b1a8c8a

Browse files
[flang][test] Fix REQUIRES and options for a few x86 specific tests (#146872)
These should have been looking for the "x86" target not "x64_64". When run on AArch64 they failed because bbc tried to compile for AArch64. Add a target option to fix that, as these tests are x86 specific.
1 parent 3e934dd commit b1a8c8a

File tree

3 files changed

+6
-6
lines changed

3 files changed

+6
-6
lines changed

flang/test/Lower/Intrinsics/product.f90

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
! REQUIRES: x86_64-registered-target
2-
! RUN: bbc --use-desc-for-alloc=false -emit-fir -hlfir=false %s -o - | FileCheck %s
1+
! REQUIRES: x86-registered-target
2+
! RUN: bbc -target x86_64-unknown-linux-gnu --use-desc-for-alloc=false -emit-fir -hlfir=false %s -o - | FileCheck %s
33

44
! CHECK-LABEL: func @_QPproduct_test(
55
! CHECK-SAME: %[[arg0:.*]]: !fir.box<!fir.array<?xi32>>{{.*}}) -> i32

flang/test/Lower/Intrinsics/sum.f90

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
! REQUIRES: x86_64-registered-target
2-
! RUN: bbc --use-desc-for-alloc=false -emit-fir -hlfir=false %s -o - | FileCheck %s
1+
! REQUIRES: x86-registered-target
2+
! RUN: bbc -target x86_64-unknown-linux-gnu --use-desc-for-alloc=false -emit-fir -hlfir=false %s -o - | FileCheck %s
33

44
! CHECK-LABEL: func @_QPsum_test(
55
! CHECK-SAME: %[[arg0:.*]]: !fir.box<!fir.array<?xi32>>{{.*}}) -> i32 {

flang/test/Lower/complex-operations.f90

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
! REQUIRES: flang-supports-f128-math
2-
! REQUIRES: x86_64-registered-target
3-
! RUN: bbc -hlfir=false %s -o - | FileCheck %s
2+
! REQUIRES: x86-registered-target
3+
! RUN: bbc -target x86_64-unknown-linux-gnu -hlfir=false %s -o - | FileCheck %s
44

55
! CHECK-LABEL: @_QPadd_test
66
subroutine add_test(a,b,c)

0 commit comments

Comments
 (0)