Mesa (master): i965: Set SWIZZLE_NOOP for unused texture units in the program keys.

Kenneth Graunke kwg at kemper.freedesktop.org
Sun Aug 26 05:57:06 UTC 2012


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

Author: Kenneth Graunke <kenneth at whitecape.org>
Date:   Tue Aug 14 11:05:55 2012 -0700

i965: Set SWIZZLE_NOOP for unused texture units in the program keys.

Previously, we left the swizzle key field as zero for unused texture
units.  The precompile sets all of them to SWIZZLE_NOOP, which meant
that we mismatched almost every time.

Since either works equally well, change it to SWIZZLE_NOOP to match
the precompiles.

Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
Reviewed-by: Paul Berry <stereotype441 at gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Eric Anholt <eric at anholt.net>

---

 src/mesa/drivers/dri/i965/brw_wm.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_wm.c b/src/mesa/drivers/dri/i965/brw_wm.c
index 6e5163b..ba6f874 100644
--- a/src/mesa/drivers/dri/i965/brw_wm.c
+++ b/src/mesa/drivers/dri/i965/brw_wm.c
@@ -437,6 +437,8 @@ brw_populate_sampler_prog_key_data(struct gl_context *ctx,
 				   struct brw_sampler_prog_key_data *key)
 {
    for (int i = 0; i < BRW_MAX_TEX_UNIT; i++) {
+      key->swizzles[i] = SWIZZLE_NOOP;
+
       if (!prog->TexturesUsed[i])
 	 continue;
 
@@ -512,9 +514,6 @@ brw_populate_sampler_prog_key_data(struct gl_context *ctx,
 	       key->gl_clamp_mask[2] |= 1 << i;
 	 }
       }
-      else {
-	 key->swizzles[i] = SWIZZLE_NOOP;
-      }
    }
 }
 




More information about the mesa-commit mailing list