From a53d4845bbf40aec8df7fd2b2d4f3e3905069e2a Mon Sep 17 00:00:00 2001 From: "Larsen, Steffen" Date: Tue, 12 Aug 2025 07:53:15 -0700 Subject: [PATCH 1/2] [CI][NFC] Clarify the breaking of the opportunistic_group This clarifies the break of sycl-rel 6.2 opportunistic_group test due to the removal of the corresponding device aspect in https://github.com/intel/llvm/pull/19238 following the refactoring of the experimental extension. Signed-off-by: Larsen, Steffen --- devops/compat_ci_exclude.sycl-rel-6_2 | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/devops/compat_ci_exclude.sycl-rel-6_2 b/devops/compat_ci_exclude.sycl-rel-6_2 index d34b3ab81ab1..9c33a0600357 100644 --- a/devops/compat_ci_exclude.sycl-rel-6_2 +++ b/devops/compat_ci_exclude.sycl-rel-6_2 @@ -29,7 +29,10 @@ KernelAndProgram/disable-caching.cpp # binaries built with sanitizers? Sanitizer -# https://github.com/intel/llvm/pull/19238 +# https://github.com/intel/llvm/pull/19238 removed the device aspect +# corresponding to the opportunistic_group support, in favor of it being merged +# into the fragment_group aspect. Since this was a full refactoring and renaming +# of an experimental extension, we accept the breaking change. NonUniformGroups/opportunistic_group.cpp # https://github.com/intel/llvm/pull/17955, experimental extension From 59b679ee2e5486a708776ca16a0bc1d1ec8e38f6 Mon Sep 17 00:00:00 2001 From: "Larsen, Steffen" Date: Tue, 12 Aug 2025 21:23:41 -0700 Subject: [PATCH 2/2] Move clarification Signed-off-by: Larsen, Steffen --- devops/compat_ci_exclude.sycl-rel-6_2 | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/devops/compat_ci_exclude.sycl-rel-6_2 b/devops/compat_ci_exclude.sycl-rel-6_2 index 9c33a0600357..efa05e65a0b9 100644 --- a/devops/compat_ci_exclude.sycl-rel-6_2 +++ b/devops/compat_ci_exclude.sycl-rel-6_2 @@ -12,6 +12,12 @@ DiscardEvents/invalid_event_exceptions.cpp # Throw exception instead of returning garbage Basic/info.cpp +# https://github.com/intel/llvm/pull/19238 removed the device aspect +# corresponding to the opportunistic_group support, in favor of it being merged +# into the fragment_group aspect. Since this was a full refactoring and renaming +# of an experimental extension, we accept the breaking change. +NonUniformGroups/opportunistic_group.cpp + # Likely OK, but need author to provide justification, get approval/confirmation # from someone: @@ -29,12 +35,6 @@ KernelAndProgram/disable-caching.cpp # binaries built with sanitizers? Sanitizer -# https://github.com/intel/llvm/pull/19238 removed the device aspect -# corresponding to the opportunistic_group support, in favor of it being merged -# into the fragment_group aspect. Since this was a full refactoring and renaming -# of an experimental extension, we accept the breaking change. -NonUniformGroups/opportunistic_group.cpp - # https://github.com/intel/llvm/pull/17955, experimental extension AsyncAlloc/device/async_alloc_from_pool.cpp AsyncAlloc/device/async_alloc_zero_init.cpp