Skip to content

Commit dc9b235

Browse files
Change patchGlobalOffset in l0 kernel to void
Signed-off-by: Dominik Dabek <dominik.dabek@intel.com>
1 parent ddcd2a5 commit dc9b235

File tree

4 files changed

+5
-6
lines changed

4 files changed

+5
-6
lines changed

level_zero/core/source/kernel/kernel.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ struct Kernel : _ze_kernel_handle_t, virtual NEO::DispatchKernelEncoderI {
114114

115115
virtual uint32_t *getGlobalOffsets() = 0;
116116
virtual ze_result_t setGlobalOffsetExp(uint32_t offsetX, uint32_t offsetY, uint32_t offsetZ) = 0;
117-
virtual uint32_t patchGlobalOffset() = 0;
117+
virtual void patchGlobalOffset() = 0;
118118

119119
virtual void patchWorkDim(uint32_t groupCountX, uint32_t groupCountY, uint32_t groupCountZ) = 0;
120120

level_zero/core/source/kernel/kernel_imp.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -883,10 +883,10 @@ ze_result_t KernelImp::setGlobalOffsetExp(uint32_t offsetX,
883883
return ZE_RESULT_SUCCESS;
884884
}
885885

886-
uint32_t KernelImp::patchGlobalOffset() {
886+
void KernelImp::patchGlobalOffset() {
887887
const NEO::KernelDescriptor &desc = kernelImmData->getDescriptor();
888888
auto dst = ArrayRef<uint8_t>(crossThreadData.get(), crossThreadDataSize);
889-
return NEO::patchVecNonPointer(dst, desc.payloadMappings.dispatchTraits.globalWorkOffset, this->globalOffsets);
889+
NEO::patchVecNonPointer(dst, desc.payloadMappings.dispatchTraits.globalWorkOffset, this->globalOffsets);
890890
}
891891

892892
void KernelImp::patchWorkDim(uint32_t groupCountX, uint32_t groupCountY, uint32_t groupCountZ) {

level_zero/core/source/kernel/kernel_imp.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ struct KernelImp : Kernel {
124124
return this->globalOffsets;
125125
}
126126
ze_result_t setGlobalOffsetExp(uint32_t offsetX, uint32_t offsetY, uint32_t offsetZ) override;
127-
uint32_t patchGlobalOffset() override;
127+
void patchGlobalOffset() override;
128128

129129
void patchWorkDim(uint32_t groupCountX, uint32_t groupCountY, uint32_t groupCountZ) override;
130130

level_zero/core/test/unit_tests/sources/kernel/test_kernel.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1814,8 +1814,7 @@ HWTEST_F(KernelGlobalWorkOffsetTests, whenSettingGlobalOffsetThenCrossThreadData
18141814
EXPECT_EQ(ZE_RESULT_SUCCESS, res);
18151815

18161816
KernelImp *kernelImp = static_cast<KernelImp *>(kernel);
1817-
uint32_t patchedCount = kernelImp->patchGlobalOffset();
1818-
EXPECT_EQ(patchedCount, 3u);
1817+
kernelImp->patchGlobalOffset();
18191818

18201819
const NEO::KernelDescriptor &desc = kernelImp->getImmutableData()->getDescriptor();
18211820
auto dst = ArrayRef<const uint8_t>(kernelImp->getCrossThreadData(), kernelImp->getCrossThreadDataSize());

0 commit comments

Comments
 (0)