Skip to content

Commit

Permalink
[SYCL][HIP] Remove unsupported from O0 tests on AMD (intel#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
2 people authored and ianayl committed Jun 13, 2024
1 parent f732ef3 commit 089dafc
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 089dafc

Please sign in to comment.