Skip to content

Commit 955047d

Browse files
[SYCL][E2E] Replace REQUIRES with XFAIL (#19955)
Those tests should not be launched only on CUDA, because they already have an aspect requirement. However, they do not pass on L0 yet because of missing functionality there. They don't pass on HIP either - still `XFAIL` is a better approach than `REQUIRES`
1 parent 6957433 commit 955047d

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

sycl/test-e2e/bindless_images/copies/host_to_host_pitched.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
// REQUIRES: aspect-ext_oneapi_bindless_images
22
// REQUIRES: aspect-ext_oneapi_bindless_images_2d_usm
3-
// REQUIRES: cuda
3+
// XFAIL: level_zero
4+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/17663
5+
// XFAIL: hip
6+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/19957
47

58
// RUN: %{build} -o %t.out
69
// RUN: %{run} %t.out

sycl/test-e2e/bindless_images/sampling_2D_USM_host.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
1-
// REQUIRES: cuda
21
// REQUIRES: aspect-ext_oneapi_bindless_images_2d_usm
2+
// XFAIL: level_zero
3+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/17663
4+
// XFAIL: hip
5+
// XFAIL-TRACKER: https://github.com/intel/llvm/issues/19957
36

47
// RUN: %{build} -o %t.out
58
// RUN: %{run-unfiltered-devices} %t.out

0 commit comments

Comments
 (0)