i965: scale factor changes should trigger recompile
authorLionel Landwerlin <lionel.g.landwerlin@intel.com>
Fri, 15 Feb 2019 13:47:48 +0000 (13:47 +0000)
committerLionel Landwerlin <lionel.g.landwerlin@intel.com>
Mon, 18 Feb 2019 12:18:13 +0000 (12:18 +0000)
Found by inspection.

Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Fixes: 3da858a6b990c5 ("intel/compiler: add scale_factors to sampler_prog_key_data")
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
src/mesa/drivers/dri/i965/brw_context.h
src/mesa/drivers/dri/i965/brw_wm.c

index 66fe5b3a8a056fcbc70d4065be183384e7bcb30b..51c3196e66a80a4205addfa7f4862b2052f974d8 100644 (file)
@@ -1460,6 +1460,16 @@ key_debug(struct brw_context *brw, const char *name, int a, int b)
    return false;
 }
 
+static inline bool
+key_debug_float(struct brw_context *brw, const char *name, float a, float b)
+{
+   if (a != b) {
+      perf_debug("  %s %f->%f\n", name, a, b);
+      return true;
+   }
+   return false;
+}
+
 void brwInitFragProgFuncs( struct dd_function_table *functions );
 
 void brw_get_scratch_bo(struct brw_context *brw,
index 9f88d625d63d60aa1e70e575a9e380fb0efa4932..3cd8f18e794c0d9531a39e11be37120ab39a9344 100644 (file)
@@ -271,12 +271,17 @@ brw_debug_recompile_sampler_key(struct brw_context *brw,
                       old_key->ayuv_image_mask,
                       key->ayuv_image_mask);
 
-
    for (unsigned int i = 0; i < MAX_SAMPLERS; i++) {
       found |= key_debug(brw, "textureGather workarounds",
                          old_key->gen6_gather_wa[i], key->gen6_gather_wa[i]);
    }
 
+   for (unsigned int i = 0; i < MAX_SAMPLERS; i++) {
+      found |= key_debug_float(brw, "scale factor",
+                               old_key->scale_factors[i],
+                               key->scale_factors[i]);
+   }
+
    return found;
 }