Skip to content

[SYCL][ESIMD][E2E] Enable some tests #14330

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 2, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,6 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// TODO This test freezed on Linux OS, enable test on linux once this issue will
// be fixed.
// REQUIRES: windows
// RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out
// RUN: %{run} %t.out
//
Expand Down
12 changes: 5 additions & 7 deletions sycl/test-e2e/ESIMD/api/replicate_smoke.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// https://github.com/intel/llvm/issues/10369
// UNSUPPORTED: gpu
//
// UNSUPPORTED: gpu-intel-pvc
// TODO: remove fno-fast-math option once the issue is investigated and the test
Expand All @@ -17,16 +15,13 @@
//
// The test checks main functionality of the esimd::replicate_vs_w_hs function.

// Temporarily disable while the failure is being investigated.
// UNSUPPORTED: windows

#include "../esimd_test_utils.hpp"

using namespace sycl;
using namespace sycl::ext::intel::esimd;
using bfloat16 = sycl::ext::oneapi::bfloat16;
using tfloat32 = sycl::ext::intel::experimental::esimd::tfloat32;

namespace syclex = sycl::ext::oneapi::experimental;
template <class T> struct char_to_int {
using type = typename std::conditional<
sizeof(T) == 1,
Expand Down Expand Up @@ -182,6 +177,8 @@ int main(int argc, char **argv) {
queue q(esimd_test::ESIMDSelector, esimd_test::createExceptionHandler());
auto dev = q.get_device();
const bool doublesSupported = dev.has(sycl::aspect::fp64);
syclex::architecture arch =
dev.get_info<syclex::info::device::architecture>();
std::cout << "Running on " << dev.get_info<sycl::info::device::name>()
<< "\n";
bool passed = true;
Expand All @@ -192,7 +189,8 @@ int main(int argc, char **argv) {
passed &= test<short>(q);
passed &= test<unsigned short>(q);
passed &= test<int>(q);
passed &= test<uint64_t>(q);
if (arch > syclex::architecture::intel_gpu_dg1)
passed &= test<uint64_t>(q);
passed &= test<float>(q);
#ifdef USE_TF32
passed &= test<tfloat32>(q);
Expand Down
3 changes: 0 additions & 3 deletions sycl/test-e2e/ESIMD/api/replicate_smoke_pvc.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,6 @@
// The test checks main functionality of the esimd::replicate_vs_w_hs function.
// PVC variant of the test - adds tfloat32.

// Temporarily disable while the failure is being investigated.
// UNSUPPORTED: windows

#define USE_TF32

#include "replicate_smoke.cpp"
4 changes: 0 additions & 4 deletions sycl/test-e2e/ESIMD/fp_in_phi.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
//
//===----------------------------------------------------------------------===//
// On Windows vector compute backend (as a part of IGC) uses llvm-7 and llvm-7
// based spirv translator. This test should start working on Windows when the
// llvm version is switched to 9.
// UNSUPPORTED: windows
// RUN: %{build} -Xclang -fsycl-allow-func-ptr -o %t.out
// RUN: %{run} %t.out
// UNSUPPORTED: ze_debug
Expand Down
6 changes: 0 additions & 6 deletions sycl/test-e2e/ESIMD/fp_in_select.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,14 +5,8 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// On Windows vector compute backend (as a part of IGC) uses llvm-7 and llvm-7
// based spirv translator. This test should start working on Windows when the
// llvm version is switched to 9.
// UNSUPPORTED: windows
// RUN: %{build} -Xclang -fsycl-allow-func-ptr -o %t.out
// RUN: %{run} %t.out
// The test fails on JITing due to use of too many registers
// REQUIRES: TEMPORARY_DISBLED
//
// The test checks that ESIMD kernels correctly handle function pointers as
// arguments of LLVM's select function.
Expand Down
2 changes: 0 additions & 2 deletions sycl/test-e2e/ESIMD/noinline_call_from_func.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@
// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
//
//===----------------------------------------------------------------------===//
// Test currently timeouts on Windows Level Zero and OpenCL
// UNSUPPORTED: windows
// RUN: %{build} -o %t.out
// RUN: env IGC_FunctionControl=3 IGC_ForceInlineStackCallWithImplArg=1 %{run} %t.out
//
Expand Down
Loading