From 522d4c61c08972103c3470e852a27ccc759a835b Mon Sep 17 00:00:00 2001 From: "Agarwal, Udit" Date: Thu, 9 Jan 2025 10:50:33 -0800 Subject: [PATCH 1/5] [SYCL][E2E] Fix tests disabled during PVC enablement in CI --- sycl/test-e2e/Basic/accessor/host_task_accessor_deduction.cpp | 4 ---- sycl/test-e2e/DeprecatedFeatures/set_arg_interop.cpp | 4 ++-- sycl/test-e2e/ESIMD/named_barriers/loop.cpp | 4 ---- sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp | 4 ---- sycl/test-e2e/GroupAlgorithm/reduce_sycl2020.cpp | 4 ---- 5 files changed, 2 insertions(+), 18 deletions(-) diff --git a/sycl/test-e2e/Basic/accessor/host_task_accessor_deduction.cpp b/sycl/test-e2e/Basic/accessor/host_task_accessor_deduction.cpp index e6da8d355a0f6..597e4ee57248d 100644 --- a/sycl/test-e2e/Basic/accessor/host_task_accessor_deduction.cpp +++ b/sycl/test-e2e/Basic/accessor/host_task_accessor_deduction.cpp @@ -1,6 +1,2 @@ // RUN: %{build} -Daccessor_new_api_test %S/Inputs/host_task_accessor.cpp -o %t.out // RUN: %{run} %t.out - -// Disabled on PVC without igc-dev due to timeout. -// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev -// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 diff --git a/sycl/test-e2e/DeprecatedFeatures/set_arg_interop.cpp b/sycl/test-e2e/DeprecatedFeatures/set_arg_interop.cpp index 18388bce6c289..325a329a916dd 100644 --- a/sycl/test-e2e/DeprecatedFeatures/set_arg_interop.cpp +++ b/sycl/test-e2e/DeprecatedFeatures/set_arg_interop.cpp @@ -1,7 +1,7 @@ // REQUIRES: opencl, opencl_icd -// XFAIL: arch-intel_gpu_pvc -// XFAIL-TRACKER: https://github.com/intel/llvm/issues/14826 +// UNSUPPORTED: arch-intel_gpu_pvc +// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/16576 // RUN: %{build} -D__SYCL_INTERNAL_API -o %t.out %opencl_lib -O3 // RUN: %{run} %t.out diff --git a/sycl/test-e2e/ESIMD/named_barriers/loop.cpp b/sycl/test-e2e/ESIMD/named_barriers/loop.cpp index 9b526125153c7..e56ddfe5ad0ae 100644 --- a/sycl/test-e2e/ESIMD/named_barriers/loop.cpp +++ b/sycl/test-e2e/ESIMD/named_barriers/loop.cpp @@ -10,10 +10,6 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out -// Disabled on PVC without igc-dev due to flaky failures. -// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev -// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 - // Test checks support of named barrier in a loop in ESIMD kernel. // SLM and surface size is 32 bytes, 16 bytes per iteration. // Each iteration has 1 barrier and 1 producer. Producer stores data to SLM, diff --git a/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp b/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp index 35053824ed2af..b42c58181ca4c 100644 --- a/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp +++ b/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp @@ -10,10 +10,6 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out -// Disabled on PVC due to flaky failures. -// UNSUPPORTED: arch-intel_gpu_pvc -// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 - // Test checks support of named barrier in a loop in ESIMD kernel. // First iteration has 1 barrier and 1 producer, second - 2 barriers and 2 // producers. Producer stores data to SLM, then all threads read SLM and store diff --git a/sycl/test-e2e/GroupAlgorithm/reduce_sycl2020.cpp b/sycl/test-e2e/GroupAlgorithm/reduce_sycl2020.cpp index 67b29d213032b..321dc76923fb3 100644 --- a/sycl/test-e2e/GroupAlgorithm/reduce_sycl2020.cpp +++ b/sycl/test-e2e/GroupAlgorithm/reduce_sycl2020.cpp @@ -1,10 +1,6 @@ // RUN: %{build} -fsycl-device-code-split=per_kernel -I . -o %t.out // RUN: %{run} %t.out -// Disabled on PVC without igc-dev due to timeout. -// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev -// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 - #include "support.h" #include From 983fb43fb6fcfc3a303b80aa1109ac3c8b44c496 Mon Sep 17 00:00:00 2001 From: "Agarwal, Udit" Date: Thu, 9 Jan 2025 15:49:43 -0800 Subject: [PATCH 2/5] Disable loop_extended.cpp again --- sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp b/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp index b42c58181ca4c..35053824ed2af 100644 --- a/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp +++ b/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp @@ -10,6 +10,10 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out +// Disabled on PVC due to flaky failures. +// UNSUPPORTED: arch-intel_gpu_pvc +// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 + // Test checks support of named barrier in a loop in ESIMD kernel. // First iteration has 1 barrier and 1 producer, second - 2 barriers and 2 // producers. Producer stores data to SLM, then all threads read SLM and store From 8490ac499dcb5f22ca5b9cf0cf7abce957a8709f Mon Sep 17 00:00:00 2001 From: "Agarwal, Udit" Date: Thu, 9 Jan 2025 16:32:40 -0800 Subject: [PATCH 3/5] Disable named_barriers/loop.cpp --- sycl/test-e2e/ESIMD/named_barriers/loop.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/sycl/test-e2e/ESIMD/named_barriers/loop.cpp b/sycl/test-e2e/ESIMD/named_barriers/loop.cpp index e56ddfe5ad0ae..9b526125153c7 100644 --- a/sycl/test-e2e/ESIMD/named_barriers/loop.cpp +++ b/sycl/test-e2e/ESIMD/named_barriers/loop.cpp @@ -10,6 +10,10 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out +// Disabled on PVC without igc-dev due to flaky failures. +// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev +// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 + // Test checks support of named barrier in a loop in ESIMD kernel. // SLM and surface size is 32 bytes, 16 bytes per iteration. // Each iteration has 1 barrier and 1 producer. Producer stores data to SLM, From bcd59e65b0c1f00268d3d77783fbf2ab72f37f0a Mon Sep 17 00:00:00 2001 From: "Agarwal, Udit" Date: Fri, 10 Jan 2025 10:43:39 -0800 Subject: [PATCH 4/5] Revert "Disable named_barriers/loop.cpp" This reverts commit 8490ac499dcb5f22ca5b9cf0cf7abce957a8709f. --- sycl/test-e2e/ESIMD/named_barriers/loop.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/sycl/test-e2e/ESIMD/named_barriers/loop.cpp b/sycl/test-e2e/ESIMD/named_barriers/loop.cpp index 9b526125153c7..e56ddfe5ad0ae 100644 --- a/sycl/test-e2e/ESIMD/named_barriers/loop.cpp +++ b/sycl/test-e2e/ESIMD/named_barriers/loop.cpp @@ -10,10 +10,6 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out -// Disabled on PVC without igc-dev due to flaky failures. -// UNSUPPORTED: arch-intel_gpu_pvc && !igc-dev -// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 - // Test checks support of named barrier in a loop in ESIMD kernel. // SLM and surface size is 32 bytes, 16 bytes per iteration. // Each iteration has 1 barrier and 1 producer. Producer stores data to SLM, From 032a0c3c60e7951e03836fc15db65f170fb111f9 Mon Sep 17 00:00:00 2001 From: "Agarwal, Udit" Date: Fri, 10 Jan 2025 10:43:57 -0800 Subject: [PATCH 5/5] Revert "Disable loop_extended.cpp again" This reverts commit 983fb43fb6fcfc3a303b80aa1109ac3c8b44c496. --- sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp | 4 ---- 1 file changed, 4 deletions(-) diff --git a/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp b/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp index 35053824ed2af..b42c58181ca4c 100644 --- a/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp +++ b/sycl/test-e2e/ESIMD/named_barriers/loop_extended.cpp @@ -10,10 +10,6 @@ // RUN: %{build} -o %t.out // RUN: %{run} %t.out -// Disabled on PVC due to flaky failures. -// UNSUPPORTED: arch-intel_gpu_pvc -// UNSUPPORTED-TRACKER: https://github.com/intel/llvm/issues/14826 - // Test checks support of named barrier in a loop in ESIMD kernel. // First iteration has 1 barrier and 1 producer, second - 2 barriers and 2 // producers. Producer stores data to SLM, then all threads read SLM and store