Skip to content

Commit

Permalink
[SYCL][HIP] Remove unsupported from O0 tests on AMD
Browse files Browse the repository at this point in the history
This was tracked down to a bug in ROCm that seems to be fixed with newer
versions, and the CI is now on ROCm 6+ so these should be fine.

ROCm ticket: ROCm/clr#13

The reduce over group test works on W6800 and MI210, but it seems for
gfx1031 it reports not supporting shared USM, note that HIP for gfx1031
isn't officially supported by AMD.
  • Loading branch information
npmiller committed May 30, 2024
1 parent 8a33df3 commit 8890b87
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
// Test hangs on AMD with https://github.com/intel/llvm/pull/8412
// UNSUPPORTED: hip_amd

// Windows doesn't yet have full shutdown().
// UNSUPPORTED: ze_debug && windows
//
// The CI GPU doesn't report support for shared USM
// XFAIL: hip_amd

// RUN: %{build} -o %t.out
// RUN: %{run} %t.out
Expand Down
4 changes: 0 additions & 4 deletions sycl/test-e2e/HierPar/hier_par_wgscope_O0.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//

// Test hangs on AMD with https://github.com/intel/llvm/pull/8412
// UNSUPPORTED: hip_amd

// RUN: %{build} -O0 -o %t.out

// RUN: %{run} %t.out
Expand Down
3 changes: 0 additions & 3 deletions sycl/test-e2e/Regression/unoptimized_stream.cpp
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
// Test hangs on AMD with https://github.com/intel/llvm/pull/8412
// UNSUPPORTED: hip_amd

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

Expand Down

0 comments on commit 8890b87

Please sign in to comment.