i965: Fix 8-wide FB writes on gen6.
authorEric Anholt <eric@anholt.net>
Sun, 22 Aug 2010 07:47:45 +0000 (00:47 -0700)
committerEric Anholt <eric@anholt.net>
Sun, 22 Aug 2010 07:52:18 +0000 (00:52 -0700)
My merge of Zhenyu's patch on top of my previous patches broke it by
my code expecting simd16 single write and Zhenyu's simd8 path being
disabled by mine.  Merge the two for success.

src/mesa/drivers/dri/i965/brw_wm_emit.c

index 05b440eaa8389929048877caade38ee595deb158..f3ad01b3fec3b06b08f798b44c12c62835fdef6d 100644 (file)
@@ -1312,7 +1312,6 @@ void emit_fb_write(struct brw_wm_compile *c,
    struct intel_context *intel = &brw->intel;
    GLuint nr = 2;
    GLuint channel;
-   int step = 0;
    int base_reg; /* For gen6 fb write with no header, starting from color payload directly!. */
 
    /* Reserve a space for AA - may not be needed:
@@ -1342,7 +1341,11 @@ void emit_fb_write(struct brw_wm_compile *c,
          * m + 6: a0
          * m + 7: a1
          */
-        brw_MOV(p, brw_message_reg(nr + channel * 2), arg0[channel]);
+        if (c->dispatch_width == 16) {
+           brw_MOV(p, brw_message_reg(nr + channel * 2), arg0[channel]);
+        } else {
+           brw_MOV(p, brw_message_reg(nr + channel), arg0[channel]);
+        }
       } else if (c->dispatch_width == 16 && brw->has_compr4) {
         /* pre-gen6 SIMD16 single source DP write looks like:
          * m + 0: r0
@@ -1361,16 +1364,6 @@ void emit_fb_write(struct brw_wm_compile *c,
         brw_MOV(p,
                 brw_message_reg(nr + channel + BRW_MRF_COMPR4),
                 arg0[channel]);
-      } else if (intel->gen >= 6) {
-         brw_set_compression_control(p, BRW_COMPRESSION_NONE);
-         brw_MOV(p, brw_message_reg(nr + channel + step), arg0[channel]); 
-         if (c->dispatch_width == 16) {
-             brw_set_compression_control(p, BRW_COMPRESSION_NONE);
-             brw_MOV(p,
-                     brw_message_reg(nr + channel + step + 1),
-                     sechalf(arg0[channel]));
-             ++step;
-         }
       } else {
         /*  mov (8) m2.0<1>:ud   r28.0<8;8,1>:ud  { Align1 } */
         /*  mov (8) m6.0<1>:ud   r29.0<8;8,1>:ud  { Align1 SecHalf } */