summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPawel Pieczul <Pawel.Pieczul@intel.com>2008-07-21 10:57:20 -0700
committerIan Romanick <ian.d.romanick@intel.com>2008-07-21 10:57:20 -0700
commitb993d539a76e7f1446890a85e4b61deec4d4162d (patch)
treea20a97e791803862d72ca6cc7808dc1baca0fae7
parent77497eb73b9aa349f41f3bcb493d84610e302371 (diff)
965: Fix color clamping issues
-rw-r--r--src/mesa/drivers/dri/i965/brw_vs_emit.c26
1 files changed, 23 insertions, 3 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vs_emit.c b/src/mesa/drivers/dri/i965/brw_vs_emit.c
index 7767d1369c..8c7bc98c61 100644
--- a/src/mesa/drivers/dri/i965/brw_vs_emit.c
+++ b/src/mesa/drivers/dri/i965/brw_vs_emit.c
@@ -1160,9 +1160,29 @@ void brw_vs_emit(struct brw_vs_compile *c )
}
if (inst->DstReg.File == PROGRAM_OUTPUT
- &&inst->DstReg.Index != VERT_RESULT_HPOS
- &&c->output_regs[inst->DstReg.Index].used_in_src)
- brw_MOV(p, get_dst(c, inst->DstReg), dst);
+ && inst->DstReg.Index != VERT_RESULT_HPOS
+ && c->output_regs[inst->DstReg.Index].used_in_src) {
+ /* Result color clamping.
+ *
+ * When destination register is an output register and it's
+ * primary/secondary front/back color, we have to clamp the result
+ * to [0,1]. This is done by enabling the saturation bit for the
+ * last instruction.
+ *
+ * We don't use brw_set_saturate() as it modifies
+ * p->current->header.saturate, which affects all the subsequent
+ * instructions. Instead, we directly modify the header of the last
+ * (already stored) instruction.
+ */
+ if (inst->DstReg.File == PROGRAM_OUTPUT) {
+ if ((inst->DstReg.Index == VERT_RESULT_COL0) ||
+ (inst->DstReg.Index == VERT_RESULT_COL1) ||
+ (inst->DstReg.Index == VERT_RESULT_BFC0) ||
+ (inst->DstReg.Index == VERT_RESULT_BFC1)) {
+ p->store[p->nr_insn-1].header.saturate = 1;
+ }
+ }
+ }
release_tmps(c);
}