[Mesa-dev] [PATCH 5/5] i965/vec4: Trivial copy propagate clean-up.

Francisco Jerez currojerez at riseup.net
Fri Mar 20 07:16:17 PDT 2015


Fix typo and punctuation in a comment, break long line and add space
before curly bracket.
---
 src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp
index 02fa27e..c4120b4 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp
+++ b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp
@@ -339,8 +339,8 @@ try_copy_propagate(struct brw_context *brw, vec4_instruction *inst,
       if (dst_saturate_mask != inst->dst.writemask)
          return false;
 
-      /* Limit saturate propagation only to SEL with src1 bounded within 1.0
-       * and 1.0 otherwise, skip copy propagate altogether
+      /* Limit saturate propagation only to SEL with src1 bounded within 0.0
+       * and 1.0, otherwise skip copy propagate altogether.
        */
       switch(inst->opcode) {
       case BRW_OPCODE_SEL:
@@ -452,8 +452,9 @@ vec4_visitor::opt_copy_propagation(bool do_constant_prop)
 	 for (int i = 0; i < 4; i++) {
 	    if (inst->dst.writemask & (1 << i)) {
                entries[reg].value[i] = direct_copy ? &inst->src[0] : NULL;
-               entries[reg].saturatemask |= (((inst->saturate && direct_copy) ? 1 : 0) << i);
-	    }
+               entries[reg].saturatemask |=
+                  inst->saturate && direct_copy ? 1 << i : 0;
+            }
 	 }
 
 	 /* Clear the records for any registers whose current value came from
@@ -464,7 +465,7 @@ vec4_visitor::opt_copy_propagation(bool do_constant_prop)
 	 else {
 	    for (unsigned i = 0; i < alloc.total_size; i++) {
 	       for (int j = 0; j < 4; j++) {
-		  if (is_channel_updated(inst, entries[i].value, j)){
+		  if (is_channel_updated(inst, entries[i].value, j)) {
 		     entries[i].value[j] = NULL;
 		     entries[i].saturatemask &= ~(1 << j);
                   }
-- 
2.1.3



More information about the mesa-dev mailing list