From 52066376c3155e729df5bfa8cd2210e5cd75b004 Mon Sep 17 00:00:00 2001 From: Nicolas Miller Date: Tue, 29 Apr 2025 10:02:28 +0100 Subject: [PATCH] [SYCL][E2E] Add missing sg-32 requires These tests are using `redq_sub_group_size(32)` so they should require the feature. This is visible on certain AMD GPUs that use a 64 thread sub-group size instead of 32 threads. --- sycl/test-e2e/SubGroup/attributes_cuda_hip.cpp | 1 + sycl/test-e2e/syclcompat/launch/launch_policy.cpp | 1 + 2 files changed, 2 insertions(+) diff --git a/sycl/test-e2e/SubGroup/attributes_cuda_hip.cpp b/sycl/test-e2e/SubGroup/attributes_cuda_hip.cpp index 7401640cd8901..216d0f031b014 100644 --- a/sycl/test-e2e/SubGroup/attributes_cuda_hip.cpp +++ b/sycl/test-e2e/SubGroup/attributes_cuda_hip.cpp @@ -9,6 +9,7 @@ // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception // //===----------------------------------------------------------------------===// +// REQUIRES: sg-32 #include "attributes_helper.hpp" diff --git a/sycl/test-e2e/syclcompat/launch/launch_policy.cpp b/sycl/test-e2e/syclcompat/launch/launch_policy.cpp index b63438ffb497a..e7433a4f69984 100644 --- a/sycl/test-e2e/syclcompat/launch/launch_policy.cpp +++ b/sycl/test-e2e/syclcompat/launch/launch_policy.cpp @@ -19,6 +19,7 @@ * Description: * launch with config tests **************************************************************************/ +// REQUIRES: sg-32 // RUN: %{build} -fsycl-device-code-split=per_kernel -o %t.out // RUN: %{run} %t.out