From 7319d0546933604deed055f08371e2898c5a83ec Mon Sep 17 00:00:00 2001 From: Compute-Runtime-Validation Date: Fri, 19 Aug 2022 06:18:41 +0200 Subject: [PATCH] Revert "feat(zebin): set missing pointer size for arg type buffer address" This reverts commit cc17c53a77a02257a673819875a6d8d87b4ae33a. Signed-off-by: Compute-Runtime-Validation --- shared/source/device_binary_format/zebin_decoder.cpp | 1 - .../test/unit_test/device_binary_format/zebin_decoder_tests.cpp | 1 - 2 files changed, 2 deletions(-) diff --git a/shared/source/device_binary_format/zebin_decoder.cpp b/shared/source/device_binary_format/zebin_decoder.cpp index 2f486da7db..6e11c40ff2 100644 --- a/shared/source/device_binary_format/zebin_decoder.cpp +++ b/shared/source/device_binary_format/zebin_decoder.cpp @@ -875,7 +875,6 @@ NEO::DecodeError populateArgDescriptor(const NEO::Elf::ZebinKernelMetadata::Type } auto &argAsPointer = dst.payloadMappings.explicitArgs[src.argIndex].as(true); argAsPointer.bufferOffset = src.offset; - argAsPointer.pointerSize = src.size; break; } diff --git a/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp b/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp index f2ecd8b402..c853996d41 100644 --- a/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp +++ b/shared/test/unit_test/device_binary_format/zebin_decoder_tests.cpp @@ -4671,7 +4671,6 @@ TEST(PopulateArgDescriptorCrossthreadPayload, GivenArgTypeBufferOffsetWhenOffset ASSERT_EQ(1U, programInfo.kernelInfos[0]->kernelDescriptor.payloadMappings.explicitArgs.size()); const auto &arg = programInfo.kernelInfos[0]->kernelDescriptor.payloadMappings.explicitArgs[0].as(); EXPECT_EQ(8, arg.bufferOffset); - EXPECT_EQ(4, arg.pointerSize); } TEST(PopulateArgDescriptorCrossthreadPayload, givenPureStatefulArgWithBufferAddressWhenThereIsNoStatelessAccessThenPopulatesKernelDescriptorAndArgIsPureStateful) {