refactor: return scratch ptr offset as optional

Signed-off-by: Mateusz Jablonski <mateusz.jablonski@intel.com>
This commit is contained in:
Mateusz Jablonski
2025-12-10 13:48:59 +00:00
committed by Compute-Runtime-Automation
parent ecba2675a3
commit 08bc465307
2 changed files with 4 additions and 4 deletions

View File

@@ -362,11 +362,11 @@ struct alignas(32) ImplicitArgs {
}
}
size_t getScratchPtrOffset() const {
std::optional<size_t> getScratchPtrOffset() const {
if (v1.header.structVersion == 1) {
return offsetof(ImplicitArgsV1, scratchPtr);
}
return 0;
return std::nullopt;
}
};

View File

@@ -249,7 +249,7 @@ TEST(ImplicitArgsV0Test, givenImplicitArgsV0WhenSettingFieldsThenCorrectFieldsAr
EXPECT_EQ(32u, implicitArgs.getSimdWidth().value());
EXPECT_EQ(0u, implicitArgs.getScratchPtrOffset());
EXPECT_FALSE(implicitArgs.getScratchPtrOffset().has_value());
}
TEST(ImplicitArgsV1Test, givenImplicitArgsV1WhenSettingFieldsThenCorrectFieldsAreSet) {
@@ -353,7 +353,7 @@ TEST(ImplicitArgsV2Test, givenImplicitArgsV2WhenSettingFieldsThenCorrectFieldsAr
EXPECT_EQ(0x1000123400u, implicitArgs.v2.rtGlobalBufferPtr);
EXPECT_EQ(0x1234000u, implicitArgs.v2.syncBufferPtr);
EXPECT_EQ(0u, implicitArgs.getScratchPtrOffset());
EXPECT_FALSE(implicitArgs.getScratchPtrOffset().has_value());
}
TEST(ImplicitArgsV1Test, givenImplicitArgsWithUnknownVersionWhenSettingFieldsThenFieldsAreNotPopulated) {