lee/renderdoc
1
0
Fork 0
mirror of https://github.com/baldurk/renderdoc.git synced 2026-09-07 14:25:44 +00:00
Code Issues Packages Projects Releases Wiki Activity

Tweak inlining for composite extract/construct

Browse Source
This commit is contained in:
baldurk
2016-02-07 18:42:00 +01:00
parent a91a784a8d
commit 458ad0dd4c
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
renderdoc/driver/shaders/spirv/spirv_disassemble.cpp
+10 -2
View File
@@ -1337,10 +1337,15 @@ void SPVModule::Disassemble()
if(arg->op)
{
// allow less inlining in composite constructs
int maxAllowedComplexity = NO_INLINE_COMPLEXITY;
if(instr->opcode == spv::OpCompositeConstruct)
maxAllowedComplexity = RDCMIN(2, maxAllowedComplexity);
// don't fold up too complex an operation
// allow some ops to have multiple arguments, others with many
// arguments should not be inlined
if(arg->op->complexity >= NO_INLINE_COMPLEXITY ||
if(arg->op->complexity >= maxAllowedComplexity ||
(arg->op->arguments.size() > 2 &&
arg->opcode != spv::OpAccessChain &&
arg->opcode != spv::OpSelect &&
@@ -1357,7 +1362,10 @@ void SPVModule::Disassemble()
instr->op->complexity = maxcomplex;
if(instr->opcode != spv::OpStore && instr->opcode != spv::OpLoad && instr->op->inlineArgs)
if(instr->opcode != spv::OpStore &&
instr->opcode != spv::OpLoad &&
instr->opcode != spv::OpCompositeExtract &&
instr->op->inlineArgs)
instr->op->complexity++;
// TODO this should be more sophisticated but need a 'pure' check to make sure