lee/renderdoc
1
0
Fork 0
mirror of https://github.com/baldurk/renderdoc.git synced 2026-08-24 07:26:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add OpName before OpModuleProcessed

Browse Source 
The SPIR-V spec requires OpName comes before any OpModuleProcessed.
Fixes a SPIR-V validation error on the generated PostVS shader if the
original shader has any OpModuleProcessed.
This commit is contained in:
Alex Smith
2018-12-29 14:00:34 +00:00
committed by Baldur Karlsson
parent 2ec0096bc2
commit 9db805a1cd
1 changed files with 11 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
renderdoc/driver/shaders/spirv/spirv_editor.cpp
+11 -4
View File
@@ -314,11 +314,18 @@ void SPIRVEditor::SetName(uint32_t id, const char *name)
SPIRVOperation op(spv::OpName, uintName);
size_t offset = sections[SPIRVSection::Debug].endOffset;
SPIRVIterator it;
spirv.insert(spirv.begin() + offset, op.begin(), op.end());
RegisterOp(SPIRVIterator(spirv, offset));
addWords(offset, op.size());
// OpName must be before OpModuleProcessed.
for(it = Begin(SPIRVSection::Debug); it < End(SPIRVSection::Debug); ++it)
{
if(it.opcode() == spv::OpModuleProcessed)
break;
}
spirv.insert(spirv.begin() + it.offs(), op.begin(), op.end());
RegisterOp(SPIRVIterator(spirv, it.offs()));
addWords(it.offs(), op.size());
}
void SPIRVEditor::AddDecoration(const SPIRVOperation &op)