Revert "feat(zebin): set missing pointer size for arg type buffer address"

Compute-Runtime-Validation · Compute-Runtime-Automation · commit 7319d0546933 · 2022-08-19T06:54:59.000+02:00
This reverts commit cc17c53. Signed-off-by: Compute-Runtime-Validation <compute-runtime-validation@intel.com>
diff --git 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<ArgDescPointer>(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
@@ -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<ArgDescPointer>();
     EXPECT_EQ(8, arg.bufferOffset);
-    EXPECT_EQ(4, arg.pointerSize);
 }
 
 TEST(PopulateArgDescriptorCrossthreadPayload, givenPureStatefulArgWithBufferAddressWhenThereIsNoStatelessAccessThenPopulatesKernelDescriptorAndArgIsPureStateful) {

Original file line number	Diff line number	Diff line change
`@@ -875,7 +875,6 @@ NEO::DecodeError populateArgDescriptor(const NEO::Elf::ZebinKernelMetadata::Type`
`875`	`875`	`}`
`876`	`876`	`auto &argAsPointer = dst.payloadMappings.explicitArgs[src.argIndex].as<ArgDescPointer>(true);`
`877`	`877`	`argAsPointer.bufferOffset = src.offset;`
`878`		`- argAsPointer.pointerSize = src.size;`
`879`	`878`	`break;`
`880`	`879`	`}`
`881`	`880`
Original file line number	Diff line number	Diff line change
`@@ -4671,7 +4671,6 @@ TEST(PopulateArgDescriptorCrossthreadPayload, GivenArgTypeBufferOffsetWhenOffset`
`4671`	`4671`	`ASSERT_EQ(1U, programInfo.kernelInfos[0]->kernelDescriptor.payloadMappings.explicitArgs.size());`
`4672`	`4672`	`const auto &arg = programInfo.kernelInfos[0]->kernelDescriptor.payloadMappings.explicitArgs[0].as<ArgDescPointer>();`
`4673`	`4673`	`EXPECT_EQ(8, arg.bufferOffset);`
`4674`		`- EXPECT_EQ(4, arg.pointerSize);`
`4675`	`4674`	`}`
`4676`	`4675`
`4677`	`4676`	`TEST(PopulateArgDescriptorCrossthreadPayload, givenPureStatefulArgWithBufferAddressWhenThereIsNoStatelessAccessThenPopulatesKernelDescriptorAndArgIsPureStateful) {`