diff --git a/SPIRV/GlslangToSpv.cpp b/SPIRV/GlslangToSpv.cpp index 2e2a2fe595..c03bf907dc 100755 --- a/SPIRV/GlslangToSpv.cpp +++ b/SPIRV/GlslangToSpv.cpp @@ -2833,7 +2833,18 @@ spv::Id TGlslangToSpvTraverser::createSpvVariable(const glslang::TIntermSymbol* if (glslang::IsAnonymous(name)) name = ""; - return builder.createVariable(storageClass, spvType, name); + spv::Id initializer = spv::NoResult; + + if (node->getType().getQualifier().storage == glslang::EvqUniform && + !node->getConstArray().empty()) { + int nextConst = 0; + initializer = createSpvConstantFromConstUnionArray(node->getType(), + node->getConstArray(), + nextConst, + false /* specConst */); + } + + return builder.createVariable(storageClass, spvType, name, initializer); } // Return type Id of the sampled type. diff --git a/SPIRV/SpvBuilder.cpp b/SPIRV/SpvBuilder.cpp index 8755d9eacf..d726372c12 100644 --- a/SPIRV/SpvBuilder.cpp +++ b/SPIRV/SpvBuilder.cpp @@ -1198,7 +1198,7 @@ void Builder::makeDiscard() } // Comments in header -Id Builder::createVariable(StorageClass storageClass, Id type, const char* name) +Id Builder::createVariable(StorageClass storageClass, Id type, const char* name, Id initializer) { Id pointerType = makePointer(storageClass, type); Instruction* inst = new Instruction(getUniqueId(), pointerType, OpVariable); @@ -1219,6 +1219,9 @@ Id Builder::createVariable(StorageClass storageClass, Id type, const char* name) if (name) addName(inst->getResultId(), name); + if (initializer != NoResult) + inst->addIdOperand(initializer); + return inst->getResultId(); } diff --git a/SPIRV/SpvBuilder.h b/SPIRV/SpvBuilder.h index 7c1d421b37..77a14a7bea 100644 --- a/SPIRV/SpvBuilder.h +++ b/SPIRV/SpvBuilder.h @@ -271,7 +271,7 @@ class Builder { void makeDiscard(); // Create a global or function local or IO variable. - Id createVariable(StorageClass, Id type, const char* name = 0); + Id createVariable(StorageClass, Id type, const char* name = 0, Id initializer = NoResult); // Create an intermediate with an undefined value. Id createUndefined(Id type); diff --git a/Test/baseResults/spv.uniformInitializer.frag.out b/Test/baseResults/spv.uniformInitializer.frag.out new file mode 100644 index 0000000000..b1474c6fb6 --- /dev/null +++ b/Test/baseResults/spv.uniformInitializer.frag.out @@ -0,0 +1,33 @@ +spv.uniformInitializer.frag +// Module Version 10000 +// Generated by (magic number): 80007 +// Id's are bound by 16 + + Capability Shader + 1: ExtInstImport "GLSL.std.450" + MemoryModel Logical GLSL450 + EntryPoint Fragment 4 "main" 9 + ExecutionMode 4 OriginLowerLeft + Source GLSL 450 + Name 4 "main" + Name 9 "color" + Name 14 "in_color" + Decorate 9(color) Location 0 + Decorate 14(in_color) Location 0 + 2: TypeVoid + 3: TypeFunction 2 + 6: TypeFloat 32 + 7: TypeVector 6(float) 4 + 8: TypePointer Output 7(fvec4) + 9(color): 8(ptr) Variable Output + 10: 6(float) Constant 0 + 11: 6(float) Constant 1065353216 + 12: 7(fvec4) ConstantComposite 10 11 10 11 + 13: TypePointer UniformConstant 7(fvec4) + 14(in_color): 13(ptr) Variable UniformConstant 12 + 4(main): 2 Function None 3 + 5: Label + 15: 7(fvec4) Load 14(in_color) + Store 9(color) 15 + Return + FunctionEnd diff --git a/Test/spv.uniformInitializer.frag b/Test/spv.uniformInitializer.frag new file mode 100644 index 0000000000..cdae1c1dac --- /dev/null +++ b/Test/spv.uniformInitializer.frag @@ -0,0 +1,10 @@ +#version 450 + +layout (location = 0) out vec4 color; + +layout (location = 0) uniform vec4 in_color = vec4(0.0, 1.0, 0.0, 1.0); + +void main() +{ + color = in_color; +} diff --git a/gtests/Spv.FromFile.cpp b/gtests/Spv.FromFile.cpp index 410f8ed0fe..9638d11bb6 100644 --- a/gtests/Spv.FromFile.cpp +++ b/gtests/Spv.FromFile.cpp @@ -442,6 +442,7 @@ INSTANTIATE_TEST_CASE_P( "spv.rankShift.comp", "spv.specConst.vert", "spv.OVR_multiview.vert", + "spv.uniformInitializer.frag", "spv.xfbOffsetOnStructMembersAssignment.vert", })), FileNameAsCustomTestSuffix