Skip to content

Commit 9382eff

Browse files
committed
remove CombinedSamplerSuffix. IsGLSL -> SourceLanguage.
1 parent a519aa9 commit 9382eff

File tree

1 file changed

+12
-16
lines changed

1 file changed

+12
-16
lines changed

Tests/DiligentCoreTest/src/ShaderTools/SPIRVShaderResourcesTest.cpp

Lines changed: 12 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -181,8 +181,7 @@ std::vector<unsigned int> LoadSPIRVFromGLSL(const char* FilePath, SHADER_TYPE Sh
181181
void TestSPIRVResourcesInternal(const char* FilePath,
182182
const std::vector<SPIRVShaderResourceRefAttribs>& RefResources,
183183
const std::vector<unsigned int>& SPIRV,
184-
SHADER_TYPE ShaderType = SHADER_TYPE_PIXEL,
185-
const char* CombinedSamplerSuffix = nullptr)
184+
SHADER_TYPE ShaderType = SHADER_TYPE_PIXEL)
186185
{
187186
ShaderDesc ShaderDesc;
188187
ShaderDesc.Name = "SPIRVResources test";
@@ -193,7 +192,7 @@ void TestSPIRVResourcesInternal(const char*
193192
GetRawAllocator(),
194193
SPIRV,
195194
ShaderDesc,
196-
CombinedSamplerSuffix,
195+
nullptr,
197196
false, // LoadShaderStageInputs
198197
false, // LoadUniformBufferReflection
199198
EntryPoint};
@@ -232,20 +231,19 @@ void TestSPIRVResourcesInternal(const char*
232231
void TestSPIRVResources(const char* FilePath,
233232
const std::vector<SPIRVShaderResourceRefAttribs>& RefResources,
234233
SHADER_TYPE ShaderType = SHADER_TYPE_PIXEL,
235-
const char* CombinedSamplerSuffix = nullptr,
236-
bool IsGLSL = false)
234+
SHADER_SOURCE_LANGUAGE SourceLanguage = SHADER_SOURCE_LANGUAGE_HLSL)
237235
{
238-
const auto& SPIRV = IsGLSL ? LoadSPIRVFromGLSL(FilePath, ShaderType) : LoadSPIRVFromHLSL(FilePath, ShaderType);
239-
ASSERT_FALSE(SPIRV.empty()) << (IsGLSL ?
236+
const auto& SPIRV = (SourceLanguage == SHADER_SOURCE_LANGUAGE_GLSL) ? LoadSPIRVFromGLSL(FilePath, ShaderType) : LoadSPIRVFromHLSL(FilePath, ShaderType);
237+
ASSERT_FALSE(SPIRV.empty()) << (SourceLanguage == SHADER_SOURCE_LANGUAGE_GLSL ?
240238
"Failed to compile GLSL to SPIRV with glslang: " :
241239
"Failed to compile HLSL to SPIRV with glslang: ")
242240
<< FilePath;
243241

244-
LOG_INFO_MESSAGE(IsGLSL ? "Testing with GLSL->SPIRV with glslang:\n" : "Testing with HLSL->SPIRV with glslang:\n", FilePath);
242+
LOG_INFO_MESSAGE(SourceLanguage == SHADER_SOURCE_LANGUAGE_GLSL ? "Testing with GLSL->SPIRV with glslang:\n" : "Testing with HLSL->SPIRV with glslang:\n", FilePath);
245243

246-
TestSPIRVResourcesInternal(FilePath, RefResources, SPIRV, ShaderType, CombinedSamplerSuffix);
244+
TestSPIRVResourcesInternal(FilePath, RefResources, SPIRV, ShaderType);
247245

248-
if (!IsGLSL)
246+
if (SourceLanguage == SHADER_SOURCE_LANGUAGE_HLSL)
249247
{
250248
if (SPIRVShaderResourcesTest::DXCompiler && SPIRVShaderResourcesTest::DXCompiler->IsLoaded())
251249
{
@@ -255,11 +253,11 @@ void TestSPIRVResources(const char* FilePa
255253

256254
LOG_INFO_MESSAGE("Testing with HLSL->SPIRV with DXC:\n", FilePath);
257255

258-
TestSPIRVResourcesInternal(FilePath, RefResources, SPIRV_DXC, ShaderType, CombinedSamplerSuffix);
256+
TestSPIRVResourcesInternal(FilePath, RefResources, SPIRV_DXC, ShaderType);
259257
}
260258
else
261259
{
262-
LOG_INFO_MESSAGE("HLSL->SPIRV with DXC skipped because DXCompiler is not available:\n", FilePath);
260+
LOG_INFO_MESSAGE("HLSL->SPIRV with DXC skipped because DXCompiler is not available\n");
263261
}
264262
}
265263
}
@@ -339,8 +337,7 @@ TEST_F(SPIRVShaderResourcesTest, AtomicCounters)
339337
SPIRVShaderResourceRefAttribs{"AtomicCounterBuffer", 1, SPIRVResourceType::RWStorageBuffer, RESOURCE_DIM_BUFFER, 0, 4, 0},
340338
},
341339
SHADER_TYPE_PIXEL,
342-
nullptr,
343-
true); // IsGLSL = true
340+
SHADER_SOURCE_LANGUAGE_GLSL);
344341
}
345342

346343
TEST_F(SPIRVShaderResourcesTest, InputAttachments)
@@ -361,8 +358,7 @@ TEST_F(SPIRVShaderResourcesTest, AccelerationStructures)
361358
SPIRVShaderResourceRefAttribs{"g_AccelStruct", 1, SPIRVResourceType::AccelerationStructure, RESOURCE_DIM_UNDEFINED, 0, 0, 0},
362359
},
363360
SHADER_TYPE_RAY_GEN,
364-
nullptr,
365-
true); // IsGLSL = true
361+
SHADER_SOURCE_LANGUAGE_GLSL); // IsGLSL = true
366362
}
367363

368364
TEST_F(SPIRVShaderResourcesTest, PushConstants)

0 commit comments

Comments
 (0)