Browse Source

tgsi: (trivial) only verify target for is_tex instructions

d3d10 state tracker does not encode (valid) target (only offsets are
really used from the texture bits), since that information always comes
from the sview dcl, and not the instruction (note the meaning of target
is actually slightly different between gl and d3d10 in any case, because
d3d10 target does never include shadow bit).
Also move the msaa sampler identification as well - would need to set that
on the sview not sampler, so while this does not fix it make it at least
obvious it won't work with sample instructions.
tags/12.0-branchpoint
Roland Scheidegger 9 years ago
parent
commit
2d3b8aefda
1 changed files with 7 additions and 8 deletions
  1. 7
    8
      src/gallium/auxiliary/tgsi/tgsi_scan.c

+ 7
- 8
src/gallium/auxiliary/tgsi/tgsi_scan.c View File

@@ -184,14 +184,14 @@ scan_instruction(struct tgsi_shader_info *info,
/* Texture samplers */
if (src->Register.File == TGSI_FILE_SAMPLER) {
const unsigned index = src->Register.Index;
const unsigned target = fullinst->Texture.Texture;

assert(fullinst->Instruction.Texture);
assert(index < Elements(info->is_msaa_sampler));
assert(index < PIPE_MAX_SAMPLERS);
assert(target < TGSI_TEXTURE_UNKNOWN);

if (tgsi_get_opcode_info(fullinst->Instruction.Opcode)->is_tex) {
const unsigned target = fullinst->Texture.Texture;
assert(target < TGSI_TEXTURE_UNKNOWN);
/* for texture instructions, check that the texture instruction
* target matches the previous sampler view declaration (if there
* was one.)
@@ -205,12 +205,11 @@ scan_instruction(struct tgsi_shader_info *info,
*/
assert(info->sampler_targets[index] == target);
}
}

/* MSAA samplers */
if (target == TGSI_TEXTURE_2D_MSAA ||
target == TGSI_TEXTURE_2D_ARRAY_MSAA) {
info->is_msaa_sampler[src->Register.Index] = TRUE;
/* MSAA samplers */
if (target == TGSI_TEXTURE_2D_MSAA ||
target == TGSI_TEXTURE_2D_ARRAY_MSAA) {
info->is_msaa_sampler[src->Register.Index] = TRUE;
}
}
}


Loading…
Cancel
Save