Skip to content

Commit 2ca4181

Browse files
author
devsh
committed
disable hashing redundant data
1 parent fd0f7c4 commit 2ca4181

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/nbl/video/utilities/CAssetConverter.cpp

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1129,11 +1129,12 @@ class HashVisit : public CAssetConverter::CHashCache::hash_impl_base
11291129
assert(hlsl::bitCount(stage) == 1);
11301130
hasher << stage;
11311131
hasher << arg0.requiredSubgroupSize;
1132-
if constexpr (std::is_same_v<AssetT, ICPURayTracingPipeline>)
1133-
{
1134-
const auto groupIndex = std::get<2>(argTuple);
1135-
hasher << groupIndex;
1136-
}
1132+
// Shaders in Groups are hashed in order and non-present ones calls `nullOptional` so just this info should be enough to produce unique input to hashing function
1133+
//if constexpr (std::is_same_v<AssetT, ICPURayTracingPipeline>)
1134+
//{
1135+
// const auto groupIndex = std::get<2>(argTuple);
1136+
// hasher << groupIndex;
1137+
//}
11371138
if (!arg0.entries.empty())
11381139
{
11391140
for (const auto& specConstant : arg0.entries)
@@ -1415,6 +1416,7 @@ bool CAssetConverter::CHashCache::hash_impl::operator()(lookup_t<ICPURayTracingP
14151416
if (!visitor())
14161417
return false;
14171418
const auto& params = asset->getCachedCreationParams();
1419+
hasher << params.flags;
14181420
hasher << params.maxRecursionDepth;
14191421
hasher << params.dynamicStackSize;
14201422
return true;

0 commit comments

Comments
 (0)