Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

Commit b060090

Browse files
authored
[SYCL][HIP] Update HIP AMD XFAILs (#563)
The get_backend test was missing the HIP plugin case. The ESIMD tests are not supported for HIP in general, not just on Nvidia. The subgroup built-ins were added for AMD in intel/llvm#4208. The kernel_info.cpp test was fixed in #474 and now passes with HIP. The kernel_bundle_ignore_sycl_external.cpp only fails with HIP Nvidia, not on AMD.
1 parent 7b561d5 commit b060090

16 files changed

+10
-30
lines changed

SYCL/Basic/kernel_info.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,6 @@
55
//
66
// Fail is flaky for level_zero, enable when fixed.
77
// UNSUPPORTED: level_zero
8-
//
9-
// Failing on HIP AMD and HIP Nvidia
10-
// XFAIL: hip_amd || hip_nvidia
118

129
//==--- kernel_info.cpp - SYCL kernel info test ----------------------------==//
1310
//

SYCL/Basic/linear-sub_group.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,6 @@
44
// RUN: %GPU_RUN_PLACEHOLDER %t.out
55
// RUN: %ACC_RUN_PLACEHOLDER %t.out
66
//
7-
// Missing built-ins on AMD
8-
// XFAIL: hip_amd
97
//==--------------- linear-sub_group.cpp - SYCL linear id test -------------==//
108
//
119
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.

SYCL/DeprecatedFeatures/get_backend.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ bool check(backend be) {
1919
case backend::level_zero:
2020
case backend::cuda:
2121
case backend::host:
22+
case backend::hip:
2223
return true;
2324
default:
2425
return false;

SYCL/ESIMD/api/simd_negation_operator.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88
// REQUIRES: gpu
9-
// UNSUPPORTED: cuda || hip_nvidia
9+
// UNSUPPORTED: cuda || hip
1010
// RUN: %clangxx -fsycl %s -o %t.out
1111
// RUN: %GPU_RUN_PLACEHOLDER %t.out
1212
//

SYCL/ESIMD/api/simd_view_copy_move_assign.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88
// REQUIRES: gpu
9-
// UNSUPPORTED: cuda || hip_nvidia
9+
// UNSUPPORTED: cuda || hip
1010
// RUN: %clangxx -fsycl %s -o %t.out
1111
// RUN: %GPU_RUN_PLACEHOLDER %t.out
1212
//

SYCL/ESIMD/api/simd_view_negation_operator.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88
// REQUIRES: gpu
9-
// UNSUPPORTED: cuda || hip_nvidia
9+
// UNSUPPORTED: cuda || hip
1010
// RUN: %clangxx -fsycl %s -o %t.out
1111
// RUN: %GPU_RUN_PLACEHOLDER %t.out
1212
//

SYCL/ESIMD/api/slm_gather_scatter.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// REQUIRES: gpu
2-
// UNSUPPORTED: cuda || hip_nvidia
2+
// UNSUPPORTED: cuda || hip
33
// RUN: %clangxx -fsycl %s -o %t.out
44
// RUN: %GPU_RUN_PLACEHOLDER %t.out
55
//

SYCL/ESIMD/api/slm_gather_scatter_rgba.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
// REQUIRES: gpu
2-
// UNSUPPORTED: cuda || hip_nvidia
2+
// UNSUPPORTED: cuda || hip
33
// RUN: %clangxx -fsycl %s -o %t.out
44
// RUN: %GPU_RUN_PLACEHOLDER %t.out
55
//

SYCL/ESIMD/regression/replicate_bug.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88
// REQUIRES: gpu
9-
// UNSUPPORTED: cuda || hip_nvidia
9+
// UNSUPPORTED: cuda || hip
1010
// RUN: %clangxx -fsycl %s -o %t.out
1111
// RUN: %GPU_RUN_PLACEHOLDER %t.out
1212
//

SYCL/ESIMD/usm_gather_scatter_rgba.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
//
77
//===----------------------------------------------------------------------===//
88
// REQUIRES: gpu
9-
// UNSUPPORTED: cuda || hip_nvidia
9+
// UNSUPPORTED: cuda || hip
1010
// RUN: %clangxx -fsycl %s -o %t.out
1111
// RUN: %GPU_RUN_PLACEHOLDER %t.out
1212
//

SYCL/GroupAlgorithm/SYCL2020/permute_select.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,6 @@
33
// RUN: %GPU_RUN_PLACEHOLDER %t.out
44
// RUN: %ACC_RUN_PLACEHOLDER %t.out
55
//
6-
// Missing __spirv_SubgroupId, __spirv_SubgroupMaxSize, __spirv_SubgroupShuffle*
7-
// on AMD:
8-
// XFAIL: hip_amd
9-
//
106
//==------------ permute_select.cpp -*- C++ -*-----------------------------===//
117
//
128
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.

SYCL/GroupAlgorithm/SYCL2020/shift_left_right.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,6 @@
33
// RUN: %GPU_RUN_PLACEHOLDER %t.out
44
// RUN: %ACC_RUN_PLACEHOLDER %t.out
55
//
6-
// Missing __spirv_SubgroupId, __spirv_SubgroupMaxSize, __spirv_SubgroupShuffle*
7-
// on AMD:
8-
// XFAIL: hip_amd
9-
//
106
//==------------ shift_left_right.cpp -*- C++ -*----------------------------==//
117
//
128
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.

SYCL/Regression/kernel_bundle_ignore_sycl_external.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
// RUN: %CPU_RUN_PLACEHOLDER %t.out
44
// RUN: %ACC_RUN_PLACEHOLDER %t.out
55
//
6-
// XFAIL: cuda || hip
6+
// XFAIL: cuda || hip_nvidia
77

88
#include <CL/sycl.hpp>
99

SYCL/SubGroup/common.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,6 @@
44
// RUN: %GPU_RUN_PLACEHOLDER %t.out
55
// RUN: %ACC_RUN_PLACEHOLDER %t.out
66
//
7-
// Missing __spirv_SubgroupLocalInvocationId on AMD
8-
// XFAIL: hip_amd
9-
107
//==-------------- common.cpp - SYCL sub_group common test -----*- C++ -*---==//
118
//
129
// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.

SYCL/SubGroup/generic_reduce.cpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,6 @@
44
// RUN: %CPU_RUN_PLACEHOLDER %t.out
55
// RUN: %GPU_RUN_PLACEHOLDER %t_gpu.out
66
// RUN: %ACC_RUN_PLACEHOLDER %t.out
7-
//
8-
// Missing __spirv_SubgroupShuffleXorINTEL, __spirv_SubgroupLocalInvocationId,
9-
// __spirv_SubgroupShuffleINTEL, __spirv_SubgroupShuffleXorINTEL on AMD
10-
// XFAIL: hip_amd
117

128
#include "helper.hpp"
139
#include <CL/sycl.hpp>

SYCL/SubGroup/sub_groups_sycl2020.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,10 +2,9 @@
22
// RUN: %CPU_RUN_PLACEHOLDER %t.out
33
// RUN: %GPU_RUN_PLACEHOLDER %t.out
44
//
5-
// Missing __spirv_SubgroupLocalInvocationId on AMD
65
// Assertion `!MHostPlatform && "Plugin is not available for Host."' failed on
76
// Nvidia.
8-
// XFAIL: hip_amd || hip_nvidia
7+
// XFAIL: hip_nvidia
98

109
#include <sycl/sycl.hpp>
1110

0 commit comments

Comments
 (0)