Mesa (master): i965: Set the source operand types for gen6 if/else/ endif to integer.

Eric Anholt anholt at kemper.freedesktop.org
Wed Oct 20 04:33:57 UTC 2010


Module: Mesa
Branch: master
Commit: 6ea108e7db79cb7135a8a1ef216e25381f72c225
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=6ea108e7db79cb7135a8a1ef216e25381f72c225

Author: Eric Anholt <eric at anholt.net>
Date:   Tue Oct 19 14:50:42 2010 -0700

i965: Set the source operand types for gen6 if/else/endif to integer.

I don't think this should matter, but I'm not sure, and it's
recommended by a kernel checker in fulsim.

---

 src/mesa/drivers/dri/i965/brw_eu_emit.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_eu_emit.c b/src/mesa/drivers/dri/i965/brw_eu_emit.c
index 7348924..399b99c 100644
--- a/src/mesa/drivers/dri/i965/brw_eu_emit.c
+++ b/src/mesa/drivers/dri/i965/brw_eu_emit.c
@@ -832,8 +832,8 @@ struct brw_instruction *brw_IF(struct brw_compile *p, GLuint execute_size)
    } else {
       brw_set_dest(insn, brw_imm_w(0));
       insn->bits1.branch_gen6.jump_count = 0;
-      brw_set_src0(insn, brw_null_reg());
-      brw_set_src1(insn, brw_null_reg());
+      brw_set_src0(insn, retype(brw_null_reg(), BRW_REGISTER_TYPE_D));
+      brw_set_src1(insn, retype(brw_null_reg(), BRW_REGISTER_TYPE_D));
    }
 
    insn->header.execution_size = execute_size;
@@ -897,8 +897,8 @@ struct brw_instruction *brw_ELSE(struct brw_compile *p,
    } else {
       brw_set_dest(insn, brw_imm_w(0));
       insn->bits1.branch_gen6.jump_count = 0;
-      brw_set_src0(insn, brw_null_reg());
-      brw_set_src1(insn, brw_null_reg());
+      brw_set_src0(insn, retype(brw_null_reg(), BRW_REGISTER_TYPE_D));
+      brw_set_src1(insn, retype(brw_null_reg(), BRW_REGISTER_TYPE_D));
    }
 
    insn->header.compression_control = BRW_COMPRESSION_NONE;
@@ -956,8 +956,8 @@ void brw_ENDIF(struct brw_compile *p,
 	 brw_set_src1(insn, brw_imm_d(0x0));
       } else {
 	 brw_set_dest(insn, retype(brw_vec4_grf(0,0), BRW_REGISTER_TYPE_W));
-	 brw_set_src0(insn, brw_null_reg());
-	 brw_set_src1(insn, brw_null_reg());
+	 brw_set_src0(insn, retype(brw_null_reg(), BRW_REGISTER_TYPE_D));
+	 brw_set_src1(insn, retype(brw_null_reg(), BRW_REGISTER_TYPE_D));
       }
 
       insn->header.compression_control = BRW_COMPRESSION_NONE;




More information about the mesa-commit mailing list