Skip to content

Commit

Permalink
[SYCL][HIP] Remove unsupported from O0 tests on AMD (#13967)
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.

---------

Co-authored-by: Steffen Larsen <[email protected]>
  • Loading branch information
npmiller and steffenlarsen authored Jun 7, 2024
1 parent 4c2cbc5 commit 27bd7ae
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 10 deletions.
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
// 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
// REQUIRES: aspect-usm_shared_allocations

// 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 27bd7ae

Please sign in to comment.