Fixed run-time descriptors validation

Author: Przemog1

03_DeviceSelectionAndSharedSources/app_resources/pplnLayoutMergeTest/shader_3.comp.hlsl

@@ -1,6 +1,7 @@
 #pragma wave shader_stage(compute)
 #include "common.hlsl"
 
+// breaking this shader for testing purpose, only last binding in descriptor set may be run-time sized
 [[vk::binding(0,0)]] RWByteAddressBuffer output[];
 [[vk::binding(1,0)]] RWStructuredBuffer<SomeType> asdf;
 

03_DeviceSelectionAndSharedSources/app_resources/shader.comp.hlsl

@@ -3,7 +3,7 @@
 
 struct SomeType
 {
-    uint16_t butt[6][9][45];
+    uint16_t asdf[6][9][45];
 };
 
 [[vk::constant_id(3)]] const int TEST_INT_1 = 2;
@@ -24,7 +24,7 @@ PushConstants u_pushConstants;
 [numthreads(WorkgroupSize, 1, 1)]
 void main(uint32_t3 ID : SV_DispatchThreadID)
 {
-	const uint32_t byteOffset = sizeof(uint32_t)*ID.x+TEST_INT_1+uint32_t(asdf[69].butt[4][5][0]);
+	const uint32_t byteOffset = sizeof(uint32_t)*ID.x+TEST_INT_1+uint32_t(asdf[69].asdf[4][5][0]);
 
 	output.Store<uint32_t>(byteOffset,inputs[0].Load<uint32_t>(byteOffset)+inputs[1].Load<uint32_t>(byteOffset)
 		+ uint32_t(u_pushConstants.a.x) + uint32_t(u_pushConstants.a.y) + uint32_t(u_pushConstants.b));

03_DeviceSelectionAndSharedSources/main.cpp

@@ -40,7 +40,6 @@ class DeviceSelectionAndSharedSourcesApp final : public application_templates::M
 		if (!asset_base_t::onAppInitialized(std::move(system)))
 			return false;
 
-		// TODO: run-time sized buffers are not supporten in hlsl.. do it when testing glsl
 		// Just a check that out specialization info will match
 		//if (!introspection->canSpecializationlesslyCreateDescSetFrom())
 			//return logFail("Someone changed the shader and some descriptor binding depends on a specialization constant!");
@@ -59,11 +58,11 @@ class DeviceSelectionAndSharedSourcesApp final : public application_templates::M
 		}
 
 		CSPIRVIntrospector introspector;
-		auto mainShader = this->compileShaderAndTestIntrospection("app_resources/shader.comp.hlsl", introspector);
-		auto source = mainShader.first;
-		auto mainShaderIntrospection = mainShader.second;
+		auto compiledShader = this->compileShaderAndTestIntrospection("app_resources/shader.comp.hlsl", introspector);
+		auto source = compiledShader.first;
+		auto shaderIntrospection = compiledShader.second;
 
-		//mainShaderIntrospection->debugPrint(m_logger.get());
+		//shaderIntrospection->debugPrint(m_logger.get());
 
 		// We've now skipped the manual creation of a descriptor set layout, pipeline layout
 		ICPUShader::SSpecInfo specInfo;
@@ -314,6 +313,8 @@ class DeviceSelectionAndSharedSourcesApp final : public application_templates::M
 			const CSPIRVIntrospector::CStageIntrospectionData::SParams inspctParams = { .entryPoint = "main", .shader = spirvUnspecialized };
 
 			introspection = introspector.introspect(inspctParams);
+			introspection->debugPrint(m_logger.get());
+
 			if (!introspection)
 			{
 				logFail("SPIR-V Introspection failed, probably the required SPIR-V compilation failed first!");