Skip to content

Commit 6c7b83b

Browse files
[SYCL] Fix unused parameter Werror after PR #4407 (#4453)
1 parent badc608 commit 6c7b83b

File tree

4 files changed

+3
-11
lines changed

4 files changed

+3
-11
lines changed

sycl/source/detail/program_manager/program_manager.cpp

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1190,9 +1190,7 @@ uint32_t ProgramManager::getDeviceLibReqMask(const RTDeviceBinaryImage &Img) {
11901190
// TODO consider another approach with storing the masks in the integration
11911191
// header instead.
11921192
ProgramManager::KernelArgMask ProgramManager::getEliminatedKernelArgMask(
1193-
OSModuleHandle M, const ContextImplPtr &ContextImpl,
1194-
const DeviceImplPtr &DeviceImpl, pi::PiProgram NativePrg,
1195-
const std::string &KernelName) {
1193+
OSModuleHandle M, pi::PiProgram NativePrg, const std::string &KernelName) {
11961194
// If instructed to use a spv file, assume no eliminated arguments.
11971195
if (m_UseSpvFile && M == OSUtil::ExeModuleHandle)
11981196
return {};

sycl/source/detail/program_manager/program_manager.hpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -139,13 +139,9 @@ class ProgramManager {
139139
/// within the native program.
140140
/// \param M identifies the OS module the kernel comes from (multiple OS
141141
/// modules may have kernels with the same name).
142-
/// \param Context the context associated with the kernel.
143-
/// \param Device the device associated with the context.
144142
/// \param NativePrg the PI program associated with the kernel.
145143
/// \param KernelName the name of the kernel.
146144
KernelArgMask getEliminatedKernelArgMask(OSModuleHandle M,
147-
const ContextImplPtr &ContextImpl,
148-
const DeviceImplPtr &DeviceImpl,
149145
pi::PiProgram NativePrg,
150146
const std::string &KernelName);
151147

sycl/source/detail/scheduler/commands.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2110,8 +2110,7 @@ cl_int ExecCGCommand::enqueueImp() {
21102110
!ExecKernel->MSyclKernel->isCreatedFromSource()) {
21112111
EliminatedArgMask =
21122112
detail::ProgramManager::getInstance().getEliminatedKernelArgMask(
2113-
ExecKernel->MOSModuleHandle, ContextImpl, DeviceImpl, Program,
2114-
ExecKernel->MKernelName);
2113+
ExecKernel->MOSModuleHandle, Program, ExecKernel->MKernelName);
21152114
}
21162115
if (KernelMutex != nullptr) {
21172116
// For cacheable kernels, we use per-kernel mutex

sycl/unittests/program_manager/EliminatedArgMask.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -184,8 +184,7 @@ sycl::detail::ProgramManager::KernelArgMask getKernelArgMaskFromBundle(
184184
!ExecKernel->MSyclKernel->isCreatedFromSource());
185185

186186
return sycl::detail::ProgramManager::getInstance().getEliminatedKernelArgMask(
187-
ExecKernel->MOSModuleHandle, ContextImpl, DeviceImpl, Program,
188-
ExecKernel->MKernelName);
187+
ExecKernel->MOSModuleHandle, Program, ExecKernel->MKernelName);
189188
}
190189

191190
// After both kernels are compiled ProgramManager.NativePrograms contains info

0 commit comments

Comments
 (0)