From d6f827537839fc46c03602fda2f41469a3077309 Mon Sep 17 00:00:00 2001 From: Steffen Larsen Date: Tue, 17 Sep 2024 07:56:25 +0200 Subject: [PATCH] [SYCL][E2E] Enable passing UNSUPPORTED:gpu tests (#15391) This commit enables a selection of tests with UNSUPPORTED:gpu that seems to have been fixed since their disablement. Additionally, it restricts expected failure of image_array to just Gen12, with tracker https://github.com/intel/llvm/issues/15398. --------- Signed-off-by: Larsen, Steffen --- sycl/test-e2e/Basic/image/image_array.cpp | 3 ++- sycl/test-e2e/Printf/mixed-address-space.cpp | 1 - sycl/test-e2e/Printf/percent-symbol.cpp | 3 --- sycl/test-e2e/USM/align.cpp | 9 --------- 4 files changed, 2 insertions(+), 14 deletions(-) diff --git a/sycl/test-e2e/Basic/image/image_array.cpp b/sycl/test-e2e/Basic/image/image_array.cpp index 9473a4b48edb..132ad1cb41fa 100644 --- a/sycl/test-e2e/Basic/image/image_array.cpp +++ b/sycl/test-e2e/Basic/image/image_array.cpp @@ -4,7 +4,8 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out -// UNSUPPORTED: gpu +// See https://github.com/intel/llvm/issues/15398 +// UNSUPPORTED: gpu-intel-gen12 //==------------------- image.cpp - SYCL image basic test -----------------==// // diff --git a/sycl/test-e2e/Printf/mixed-address-space.cpp b/sycl/test-e2e/Printf/mixed-address-space.cpp index 137cf771d3a8..9f089847307c 100644 --- a/sycl/test-e2e/Printf/mixed-address-space.cpp +++ b/sycl/test-e2e/Printf/mixed-address-space.cpp @@ -10,7 +10,6 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out | FileCheck %s -// UNSUPPORTED: gpu // CHECK: Constant addrspace literal // CHECK: Generic addrspace literal diff --git a/sycl/test-e2e/Printf/percent-symbol.cpp b/sycl/test-e2e/Printf/percent-symbol.cpp index 46b812c393ef..30d65698a0a4 100644 --- a/sycl/test-e2e/Printf/percent-symbol.cpp +++ b/sycl/test-e2e/Printf/percent-symbol.cpp @@ -14,9 +14,6 @@ // RUN: %{build} -o %t.constant.out -DTEST_CONSTANT_AS // RUN: %{run} %t.constant.out | FileCheck %s // -// FIXME: Enable on GPU once %% conversion is supported there -// UNSUPPORTED: gpu -// // CHECK: %c %s %d %i %o %x %X %u // CHECK-NEXT: %f %F %e %E %a %A %g %G %n %p diff --git a/sycl/test-e2e/USM/align.cpp b/sycl/test-e2e/USM/align.cpp index 4f105a079d3e..e93cf9fc3c19 100644 --- a/sycl/test-e2e/USM/align.cpp +++ b/sycl/test-e2e/USM/align.cpp @@ -1,15 +1,6 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out -// UNSUPPORTED: gpu - -// E2E tests for annotated USM allocation functions with alignment arguments -// that are not powers of 2. Note this test does not work on gpu because some -// tests expect non-templated aligned_alloc_xxx functions to return nullptr, -// e.g. when the alignment argument is not a power of 2, while they fail to do -// so when run on gpu. This maybe because the gpu runtime has different -// behavior. Therefore, GPU is unsupported until issue #12638 gets resolved. - #include #include