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

Don't combine load/stores for non-function variables

Browse Source
This commit is contained in:
baldurk
2016-02-07 18:47:16 +01:00
parent 33f3f3b6de
commit 10313415d9
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
renderdoc/driver/shaders/spirv/spirv_disassemble.cpp
+3 -1
View File
@@ -1574,11 +1574,13 @@ void SPVModule::Disassemble()
// - The Load is preceeded by precisely one Store - not 0 or 2+
// - The previous store is 'pure', ie. does not depend on any mutated variables
// so it is safe to re-order to where the Load is.
// - The variable in question is a function variable
//
// If those conditions are met then we can remove the previous store, inline it as the load
// function argument (instead of the variable), and remove the variable.
if(instr->opcode == spv::OpLoad && funcops.size() > 1)
if(instr->opcode == spv::OpLoad && funcops.size() > 1 &&
instr->op->arguments[0]->var && instr->op->arguments[0]->var->storage == spv::StorageClassFunction)
{
SPVInstruction *prevstore = NULL;
int storecount = 0;