Commit b297b14d authored by Brian Paul's avatar Brian Paul

tgsi: we don't support indirect input/output registers in SSE codegen yet

Extend the check for indirect addressing of temp regs to include
input/output regs.

Fixes failure with piglit glsl-texcoord-array.shader_test test when using
SSE codegen.
parent d4afee92
...@@ -1705,22 +1705,26 @@ emit_cmp( ...@@ -1705,22 +1705,26 @@ emit_cmp(
/** /**
* Check if inst src/dest regs use indirect addressing into temporary * Check if inst src/dest regs use indirect addressing into temporary,
* register file. * input or output register files.
*/ */
static boolean static boolean
indirect_temp_reference(const struct tgsi_full_instruction *inst) indirect_reg_reference(const struct tgsi_full_instruction *inst)
{ {
uint i; uint i;
for (i = 0; i < inst->Instruction.NumSrcRegs; i++) { for (i = 0; i < inst->Instruction.NumSrcRegs; i++) {
const struct tgsi_full_src_register *reg = &inst->Src[i]; const struct tgsi_full_src_register *reg = &inst->Src[i];
if (reg->Register.File == TGSI_FILE_TEMPORARY && if ((reg->Register.File == TGSI_FILE_TEMPORARY ||
reg->Register.File == TGSI_FILE_INPUT ||
reg->Register.File == TGSI_FILE_OUTPUT) &&
reg->Register.Indirect) reg->Register.Indirect)
return TRUE; return TRUE;
} }
for (i = 0; i < inst->Instruction.NumDstRegs; i++) { for (i = 0; i < inst->Instruction.NumDstRegs; i++) {
const struct tgsi_full_dst_register *reg = &inst->Dst[i]; const struct tgsi_full_dst_register *reg = &inst->Dst[i];
if (reg->Register.File == TGSI_FILE_TEMPORARY && if ((reg->Register.File == TGSI_FILE_TEMPORARY ||
reg->Register.File == TGSI_FILE_INPUT ||
reg->Register.File == TGSI_FILE_OUTPUT) &&
reg->Register.Indirect) reg->Register.Indirect)
return TRUE; return TRUE;
} }
...@@ -1736,7 +1740,7 @@ emit_instruction( ...@@ -1736,7 +1740,7 @@ emit_instruction(
unsigned chan_index; unsigned chan_index;
/* we can't handle indirect addressing into temp register file yet */ /* we can't handle indirect addressing into temp register file yet */
if (indirect_temp_reference(inst)) if (indirect_reg_reference(inst))
return FALSE; return FALSE;
switch (inst->Instruction.Opcode) { switch (inst->Instruction.Opcode) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment