From 452d88ee732cce603de8718fbcf5f2f9ddbeeefd Mon Sep 17 00:00:00 2001 From: GitHub Actions Date: Tue, 25 Apr 2023 03:07:30 +0000 Subject: [PATCH 1/4] [GHA] Uplift Linux GPU RT version to 23.09.25812.14 --- devops/dependencies.json | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/devops/dependencies.json b/devops/dependencies.json index 448c7ff5004a1..5c5f4de96f5f5 100644 --- a/devops/dependencies.json +++ b/devops/dependencies.json @@ -1,15 +1,15 @@ { "linux": { "compute_runtime": { - "github_tag": "22.43.24595.30", - "version": "22.43.24595.30", - "url": "https://github.com/intel/compute-runtime/releases/tag/22.43.24595.30", + "github_tag": "23.09.25812.14", + "version": "23.09.25812.14", + "url": "https://github.com/intel/compute-runtime/releases/tag/23.09.25812.14", "root": "{DEPS_ROOT}/opencl/runtime/linux/oclgpu" }, "igc": { - "github_tag": "igc-1.0.12504.5", - "version": "1.0.12504.5", - "url": "https://github.com/intel/intel-graphics-compiler/releases/tag/igc-1.0.12504.5", + "github_tag": "igc-1.0.13463.18", + "version": "1.0.13463.18", + "url": "https://github.com/intel/intel-graphics-compiler/releases/tag/igc-1.0.13463.18", "root": "{DEPS_ROOT}/opencl/runtime/linux/oclgpu" }, "cm": { @@ -19,9 +19,9 @@ "root": "{DEPS_ROOT}/opencl/runtime/linux/oclgpu" }, "level_zero": { - "github_tag": "v1.8.12", - "version": "v1.8.12", - "url": "https://github.com/oneapi-src/level-zero/releases/tag/v1.8.12", + "github_tag": "v1.10.0", + "version": "v1.10.0", + "url": "https://github.com/oneapi-src/level-zero/releases/tag/v1.10.0", "root": "{DEPS_ROOT}/opencl/runtime/linux/oclgpu" }, "tbb": { @@ -74,4 +74,4 @@ "root": "{ARCHIVE_ROOT}/comp/oclfpga/win" } } -} +} \ No newline at end of file From 641e3889ec5daafa5d368b50917ddabe0373b731 Mon Sep 17 00:00:00 2001 From: Alexey Bader Date: Wed, 14 Jun 2023 12:29:36 -0700 Subject: [PATCH 2/4] Add O1 optimizations to fix the test. --- sycl/test-e2e/Printf/char.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/sycl/test-e2e/Printf/char.cpp b/sycl/test-e2e/Printf/char.cpp index ae222e6128dea..9dae799970cc2 100644 --- a/sycl/test-e2e/Printf/char.cpp +++ b/sycl/test-e2e/Printf/char.cpp @@ -7,11 +7,16 @@ // UNSUPPORTED: hip_amd // XFAIL: cuda && windows // -// RUN: %{build} -o %t.out +// FIXME: Default optimization level is O0 i.e. no optimizations and this test +// relies on DPC++ compiler to run an optimization, which changes the addres +// space for the string literal passed to the printf. Future driver version are +// going to support an extension relaxing the address space requirement. +// Meanwhile we explictly apply O1 optimizations to pass this test case. +// RUN: %{build} -O1 -o %t.out // RUN: %{run} %t.out | FileCheck %s // FIXME: Remove dedicated constant address space testing once generic AS // support is considered stable. -// RUN: %{build} -o %t.constant.out -DTEST_CONSTANT_AS +// RUN: %{build} -O1 -o %t.constant.out -DTEST_CONSTANT_AS // RUN: %{run} %t.constant.out | FileCheck %s // // FIXME: wchar_t* is not supported on GPU From df6b8ce3cc6eed9a53e449f7b06864d874a02048 Mon Sep 17 00:00:00 2001 From: Alexey Bader Date: Wed, 14 Jun 2023 16:29:51 -0700 Subject: [PATCH 3/4] Marked Printf/char.cpp as expected to fail on Intel GPU. --- sycl/test-e2e/Printf/char.cpp | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/sycl/test-e2e/Printf/char.cpp b/sycl/test-e2e/Printf/char.cpp index 9dae799970cc2..df1819909bd55 100644 --- a/sycl/test-e2e/Printf/char.cpp +++ b/sycl/test-e2e/Printf/char.cpp @@ -5,18 +5,13 @@ // [1]: https://en.cppreference.com/w/cpp/io/c/fprintf // // UNSUPPORTED: hip_amd -// XFAIL: cuda && windows +// XFAIL: cuda && windows || ((level_zero || opencl) && gpu) // -// FIXME: Default optimization level is O0 i.e. no optimizations and this test -// relies on DPC++ compiler to run an optimization, which changes the addres -// space for the string literal passed to the printf. Future driver version are -// going to support an extension relaxing the address space requirement. -// Meanwhile we explictly apply O1 optimizations to pass this test case. -// RUN: %{build} -O1 -o %t.out +// RUN: %{build} -o %t.out // RUN: %{run} %t.out | FileCheck %s // FIXME: Remove dedicated constant address space testing once generic AS // support is considered stable. -// RUN: %{build} -O1 -o %t.constant.out -DTEST_CONSTANT_AS +// RUN: %{build} -o %t.constant.out -DTEST_CONSTANT_AS // RUN: %{run} %t.constant.out | FileCheck %s // // FIXME: wchar_t* is not supported on GPU From 301cf5c0e478440866cd938e29fd7a9366f31238 Mon Sep 17 00:00:00 2001 From: Alexey Bader Date: Wed, 14 Jun 2023 19:52:25 -0700 Subject: [PATCH 4/4] Fix xfail condition. --- sycl/test-e2e/Printf/char.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sycl/test-e2e/Printf/char.cpp b/sycl/test-e2e/Printf/char.cpp index df1819909bd55..57c47a9349eef 100644 --- a/sycl/test-e2e/Printf/char.cpp +++ b/sycl/test-e2e/Printf/char.cpp @@ -5,7 +5,7 @@ // [1]: https://en.cppreference.com/w/cpp/io/c/fprintf // // UNSUPPORTED: hip_amd -// XFAIL: cuda && windows || ((level_zero || opencl) && gpu) +// XFAIL: cuda && windows || ((level_zero || opencl) && gpu && linux) // // RUN: %{build} -o %t.out // RUN: %{run} %t.out | FileCheck %s