Revert "ZEBin: fix CTD passing when inline data is supported"

Compute-Runtime-Validation · Compute-Runtime-Automation · commit 870b324d72e8 · 2021-11-05T13:31:50.000+01:00
This reverts commit 125946e. 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
@@ -997,7 +997,6 @@ NEO::DecodeError populateKernelDescriptor(NEO::ProgramInfo &dst, NEO::Elf::Elf<N
         kernelDescriptor.kernelAttributes.hasNonKernelArgAtomic = outExperimentalProperties.hasNonKernelArgAtomic;
     }
 
-    kernelDescriptor.kernelAttributes.flags.passInlineData = true;
     kernelDescriptor.kernelAttributes.barrierCount = execEnv.barrierCount;
     kernelDescriptor.kernelAttributes.flags.requiresDisabledMidThreadPreemption = execEnv.disableMidThreadPreemption;
     kernelDescriptor.kernelAttributes.numGrfRequired = execEnv.grfCount;
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
@@ -3509,7 +3509,6 @@ TEST(PopulateKernelDescriptor, GivenValidExeuctionEnvironmentThenPopulatedKernel
     EXPECT_EQ(0U, kernelDescriptor.kernelAttributes.workgroupWalkOrder[0]);
     EXPECT_EQ(1U, kernelDescriptor.kernelAttributes.workgroupWalkOrder[1]);
     EXPECT_EQ(2U, kernelDescriptor.kernelAttributes.workgroupWalkOrder[2]);
-    EXPECT_TRUE(kernelDescriptor.kernelAttributes.flags.passInlineData);
 }
 
 TEST(PopulateArgDescriptorPerThreadPayload, GivenArgTypeLocalIdWhenOffsetIsNonZeroThenFail) {

Original file line number	Diff line number	Diff line change
`@@ -997,7 +997,6 @@ NEO::DecodeError populateKernelDescriptor(NEO::ProgramInfo &dst, NEO::Elf::Elf<N`
`997`	`997`	`kernelDescriptor.kernelAttributes.hasNonKernelArgAtomic = outExperimentalProperties.hasNonKernelArgAtomic;`
`998`	`998`	`}`
`999`	`999`
`1000`		`- kernelDescriptor.kernelAttributes.flags.passInlineData = true;`
`1001`	`1000`	`kernelDescriptor.kernelAttributes.barrierCount = execEnv.barrierCount;`
`1002`	`1001`	`kernelDescriptor.kernelAttributes.flags.requiresDisabledMidThreadPreemption = execEnv.disableMidThreadPreemption;`
`1003`	`1002`	`kernelDescriptor.kernelAttributes.numGrfRequired = execEnv.grfCount;`
Original file line number	Diff line number	Diff line change
`@@ -3509,7 +3509,6 @@ TEST(PopulateKernelDescriptor, GivenValidExeuctionEnvironmentThenPopulatedKernel`
`3509`	`3509`	`EXPECT_EQ(0U, kernelDescriptor.kernelAttributes.workgroupWalkOrder[0]);`
`3510`	`3510`	`EXPECT_EQ(1U, kernelDescriptor.kernelAttributes.workgroupWalkOrder[1]);`
`3511`	`3511`	`EXPECT_EQ(2U, kernelDescriptor.kernelAttributes.workgroupWalkOrder[2]);`
`3512`		`- EXPECT_TRUE(kernelDescriptor.kernelAttributes.flags.passInlineData);`
`3513`	`3512`	`}`
`3514`	`3513`
`3515`	`3514`	`TEST(PopulateArgDescriptorPerThreadPayload, GivenArgTypeLocalIdWhenOffsetIsNonZeroThenFail) {`