[llvm] [libsycl][Unit][NFC] Fix unused lambda capture warnings (PR #217418)
Sergey Semenov via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 19 11:14:45 PDT 2026
https://github.com/sergey-semenov updated https://github.com/llvm/llvm-project/pull/217418
>From 73adb74db09c4c111eceba764062a11a0e2816ec Mon Sep 17 00:00:00 2001
From: Sergey Semenov <sergey.semenov at intel.com>
Date: Wed, 19 Aug 2026 11:08:32 -0700
Subject: [PATCH 1/2] [libsycl]][Unit][NFC] Fix unused lambda capture warnings
---
libsycl/unittests/mock/helpers.cpp | 38 +++++++++++++++---------------
1 file changed, 19 insertions(+), 19 deletions(-)
diff --git a/libsycl/unittests/mock/helpers.cpp b/libsycl/unittests/mock/helpers.cpp
index f12e8509f1dce..c66617a707814 100644
--- a/libsycl/unittests/mock/helpers.cpp
+++ b/libsycl/unittests/mock/helpers.cpp
@@ -65,7 +65,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olGetPlatformInfoSize)
- .WillByDefault([this](ol_platform_handle_t Platform,
+ .WillByDefault([](ol_platform_handle_t Platform,
ol_platform_info_t PropName,
size_t *PropSizeRet) -> ol_result_t {
EXPECT_NE(Platform, nullptr);
@@ -154,14 +154,14 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olDestroyProgram)
- .WillByDefault([this](ol_program_handle_t Program) -> ol_result_t {
+ .WillByDefault([](ol_program_handle_t Program) -> ol_result_t {
EXPECT_NE(Program, nullptr);
mock::releaseDummyHandle(Program);
return OL_SUCCESS;
});
ON_CALL(*this, olCreateContext)
- .WillByDefault([this](size_t NumDevices, ol_device_handle_t *Devices,
+ .WillByDefault([](size_t NumDevices, ol_device_handle_t *Devices,
ol_context_handle_t *Context) -> ol_result_t {
EXPECT_GT(NumDevices, 0);
EXPECT_NE(Devices, nullptr);
@@ -179,14 +179,14 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olDestroyContext)
- .WillByDefault([this](ol_context_handle_t Context) -> ol_result_t {
+ .WillByDefault([](ol_context_handle_t Context) -> ol_result_t {
EXPECT_NE(Context, nullptr);
mock::releaseDummyHandle(Context);
return OL_SUCCESS;
});
ON_CALL(*this, olCreateProgram)
- .WillByDefault([this](ol_device_handle_t Device, const void *ProgData,
+ .WillByDefault([](ol_device_handle_t Device, const void *ProgData,
size_t ProgDataSize,
ol_program_handle_t *Program) -> ol_result_t {
EXPECT_NE(Device, nullptr);
@@ -199,7 +199,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olIsValidBinary)
- .WillByDefault([this](ol_device_handle_t Device, const void *ProgData,
+ .WillByDefault([](ol_device_handle_t Device, const void *ProgData,
size_t ProgDataSize, bool *Valid) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_NE(ProgData, nullptr);
@@ -210,7 +210,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olGetSymbol)
- .WillByDefault([this](ol_program_handle_t Program, const char *Name,
+ .WillByDefault([](ol_program_handle_t Program, const char *Name,
ol_symbol_kind_t Kind,
ol_symbol_handle_t *Symbol) -> ol_result_t {
EXPECT_NE(Program, nullptr);
@@ -223,7 +223,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olCreateQueue)
- .WillByDefault([this](ol_context_handle_t Context,
+ .WillByDefault([](ol_context_handle_t Context,
ol_device_handle_t Device,
ol_queue_handle_t *Queue) -> ol_result_t {
std::ignore = Context;
@@ -236,21 +236,21 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olDestroyQueue)
- .WillByDefault([this](ol_queue_handle_t Queue) -> ol_result_t {
+ .WillByDefault([](ol_queue_handle_t Queue) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
mock::releaseDummyHandle(Queue);
return OL_SUCCESS;
});
ON_CALL(*this, olSyncQueue)
- .WillByDefault([this](ol_queue_handle_t Queue) -> ol_result_t {
+ .WillByDefault([](ol_queue_handle_t Queue) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
std::ignore = Queue;
return OL_SUCCESS;
});
ON_CALL(*this, olWaitEvents)
- .WillByDefault([this](ol_queue_handle_t Queue, ol_event_handle_t *Events,
+ .WillByDefault([](ol_queue_handle_t Queue, ol_event_handle_t *Events,
size_t NumEvents) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
EXPECT_NE(Events, nullptr);
@@ -261,7 +261,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olCreateEvent)
- .WillByDefault([this](ol_queue_handle_t Queue, ol_event_flags_t Flags,
+ .WillByDefault([](ol_queue_handle_t Queue, ol_event_flags_t Flags,
ol_event_handle_t *Event) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
std::ignore = Flags;
@@ -272,7 +272,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olLaunchKernel)
- .WillByDefault([this](ol_queue_handle_t Queue, ol_device_handle_t Device,
+ .WillByDefault([](ol_queue_handle_t Queue, ol_device_handle_t Device,
ol_symbol_handle_t Kernel,
const ol_kernel_launch_size_args_t *LaunchSizeArgs,
const ol_kernel_launch_prop_t *Properties,
@@ -292,13 +292,13 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olSyncEvent)
- .WillByDefault([this](ol_event_handle_t Event) -> ol_result_t {
+ .WillByDefault([](ol_event_handle_t Event) -> ol_result_t {
EXPECT_NE(Event, nullptr);
return OL_SUCCESS;
});
ON_CALL(*this, olMemcpy)
- .WillByDefault([this](ol_queue_handle_t Queue, void *DstPtr,
+ .WillByDefault([](ol_queue_handle_t Queue, void *DstPtr,
ol_device_handle_t DstDevice, const void *SrcPtr,
ol_device_handle_t SrcDevice,
size_t Size) -> ol_result_t {
@@ -322,14 +322,14 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olDestroyEvent)
- .WillByDefault([this](ol_event_handle_t Event) -> ol_result_t {
+ .WillByDefault([](ol_event_handle_t Event) -> ol_result_t {
EXPECT_NE(Event, nullptr);
mock::releaseDummyHandle(Event);
return OL_SUCCESS;
});
ON_CALL(*this, olMemAlloc)
- .WillByDefault([this](ol_device_handle_t Device, ol_alloc_type_t Type,
+ .WillByDefault([](ol_device_handle_t Device, ol_alloc_type_t Type,
size_t Size, void **AllocationOut) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_NE(Type, OL_ALLOC_TYPE_HOST);
@@ -340,7 +340,7 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olMemAllocHost)
- .WillByDefault([this](ol_device_handle_t Device, size_t Size,
+ .WillByDefault([](ol_device_handle_t Device, size_t Size,
void **AllocationOut) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_GT(Size, 0);
@@ -349,7 +349,7 @@ void mock::MockLiboffload::initDefault() {
return OL_SUCCESS;
});
- ON_CALL(*this, olMemFree).WillByDefault([this](void *Address) -> ol_result_t {
+ ON_CALL(*this, olMemFree).WillByDefault([](void *Address) -> ol_result_t {
EXPECT_NE(Address, nullptr);
mock::releaseDummyHandle(Address);
return OL_SUCCESS;
>From e64c79e893fe8ba1b749bdcdae93838e318f3a40 Mon Sep 17 00:00:00 2001
From: Sergey Semenov <sergey.semenov at intel.com>
Date: Wed, 19 Aug 2026 11:14:33 -0700
Subject: [PATCH 2/2] Apply clang-format
---
libsycl/unittests/mock/helpers.cpp | 45 +++++++++++++++---------------
1 file changed, 22 insertions(+), 23 deletions(-)
diff --git a/libsycl/unittests/mock/helpers.cpp b/libsycl/unittests/mock/helpers.cpp
index c66617a707814..7334bd27803c0 100644
--- a/libsycl/unittests/mock/helpers.cpp
+++ b/libsycl/unittests/mock/helpers.cpp
@@ -66,8 +66,8 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olGetPlatformInfoSize)
.WillByDefault([](ol_platform_handle_t Platform,
- ol_platform_info_t PropName,
- size_t *PropSizeRet) -> ol_result_t {
+ ol_platform_info_t PropName,
+ size_t *PropSizeRet) -> ol_result_t {
EXPECT_NE(Platform, nullptr);
EXPECT_NE(PropSizeRet, nullptr);
@@ -162,7 +162,7 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olCreateContext)
.WillByDefault([](size_t NumDevices, ol_device_handle_t *Devices,
- ol_context_handle_t *Context) -> ol_result_t {
+ ol_context_handle_t *Context) -> ol_result_t {
EXPECT_GT(NumDevices, 0);
EXPECT_NE(Devices, nullptr);
EXPECT_NE(Context, nullptr);
@@ -187,8 +187,8 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olCreateProgram)
.WillByDefault([](ol_device_handle_t Device, const void *ProgData,
- size_t ProgDataSize,
- ol_program_handle_t *Program) -> ol_result_t {
+ size_t ProgDataSize,
+ ol_program_handle_t *Program) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_NE(ProgData, nullptr);
EXPECT_GT(ProgDataSize, 0);
@@ -200,7 +200,7 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olIsValidBinary)
.WillByDefault([](ol_device_handle_t Device, const void *ProgData,
- size_t ProgDataSize, bool *Valid) -> ol_result_t {
+ size_t ProgDataSize, bool *Valid) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_NE(ProgData, nullptr);
EXPECT_GT(ProgDataSize, 0);
@@ -211,8 +211,8 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olGetSymbol)
.WillByDefault([](ol_program_handle_t Program, const char *Name,
- ol_symbol_kind_t Kind,
- ol_symbol_handle_t *Symbol) -> ol_result_t {
+ ol_symbol_kind_t Kind,
+ ol_symbol_handle_t *Symbol) -> ol_result_t {
EXPECT_NE(Program, nullptr);
EXPECT_NE(Name, nullptr);
std::ignore = Kind;
@@ -223,9 +223,8 @@ void mock::MockLiboffload::initDefault() {
});
ON_CALL(*this, olCreateQueue)
- .WillByDefault([](ol_context_handle_t Context,
- ol_device_handle_t Device,
- ol_queue_handle_t *Queue) -> ol_result_t {
+ .WillByDefault([](ol_context_handle_t Context, ol_device_handle_t Device,
+ ol_queue_handle_t *Queue) -> ol_result_t {
std::ignore = Context;
EXPECT_NE(Device, nullptr);
EXPECT_NE(Queue, nullptr);
@@ -251,7 +250,7 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olWaitEvents)
.WillByDefault([](ol_queue_handle_t Queue, ol_event_handle_t *Events,
- size_t NumEvents) -> ol_result_t {
+ size_t NumEvents) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
EXPECT_NE(Events, nullptr);
for (size_t I = 0; I < NumEvents; ++I) {
@@ -262,7 +261,7 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olCreateEvent)
.WillByDefault([](ol_queue_handle_t Queue, ol_event_flags_t Flags,
- ol_event_handle_t *Event) -> ol_result_t {
+ ol_event_handle_t *Event) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
std::ignore = Flags;
EXPECT_NE(Event, nullptr);
@@ -273,11 +272,11 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olLaunchKernel)
.WillByDefault([](ol_queue_handle_t Queue, ol_device_handle_t Device,
- ol_symbol_handle_t Kernel,
- const ol_kernel_launch_size_args_t *LaunchSizeArgs,
- const ol_kernel_launch_prop_t *Properties,
- size_t NumArgs, void **ArgPtrs,
- const size_t *ArgSizes) -> ol_result_t {
+ ol_symbol_handle_t Kernel,
+ const ol_kernel_launch_size_args_t *LaunchSizeArgs,
+ const ol_kernel_launch_prop_t *Properties,
+ size_t NumArgs, void **ArgPtrs,
+ const size_t *ArgSizes) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
EXPECT_NE(Device, nullptr);
EXPECT_NE(Kernel, nullptr);
@@ -299,9 +298,9 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olMemcpy)
.WillByDefault([](ol_queue_handle_t Queue, void *DstPtr,
- ol_device_handle_t DstDevice, const void *SrcPtr,
- ol_device_handle_t SrcDevice,
- size_t Size) -> ol_result_t {
+ ol_device_handle_t DstDevice, const void *SrcPtr,
+ ol_device_handle_t SrcDevice,
+ size_t Size) -> ol_result_t {
EXPECT_NE(Queue, nullptr);
EXPECT_NE(DstPtr, nullptr);
EXPECT_NE(DstDevice, nullptr);
@@ -330,7 +329,7 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olMemAlloc)
.WillByDefault([](ol_device_handle_t Device, ol_alloc_type_t Type,
- size_t Size, void **AllocationOut) -> ol_result_t {
+ size_t Size, void **AllocationOut) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_NE(Type, OL_ALLOC_TYPE_HOST);
EXPECT_GT(Size, 0);
@@ -341,7 +340,7 @@ void mock::MockLiboffload::initDefault() {
ON_CALL(*this, olMemAllocHost)
.WillByDefault([](ol_device_handle_t Device, size_t Size,
- void **AllocationOut) -> ol_result_t {
+ void **AllocationOut) -> ol_result_t {
EXPECT_NE(Device, nullptr);
EXPECT_GT(Size, 0);
EXPECT_NE(AllocationOut, nullptr);
More information about the llvm-commits
mailing list