i965/gen8: Remove dead assertion
authorTopi Pohjolainen <topi.pohjolainen@intel.com>
Sun, 3 Jan 2016 13:06:09 +0000 (15:06 +0200)
committerTopi Pohjolainen <topi.pohjolainen@intel.com>
Wed, 10 Feb 2016 07:11:34 +0000 (09:11 +0200)
The assertion is inside a condition mandating num_samples > 1 and
therefore the first half of the constraint is always met. The
second half in turn would only be applicable for single sampled
case and moreover it is trying to falsely check against surface
type instead of format.
Subsequent patches will introduce proper support for the lossless
compression and dropping this here makes the patches a little
simpler.

Signed-off-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
Reviewed-by: Ben Widawsky <benjamin.widawsky@intel.com>
src/mesa/drivers/dri/i965/gen8_surface_state.c

index 0df25d2557c1468c59293dd57339d0449b22ac8d..fc8f701a24c196db52a7272c99e8526b5866eb18 100644 (file)
@@ -243,12 +243,6 @@ gen8_emit_texture_surface_state(struct brw_context *brw,
        */
       if (brw->gen >= 9 || mt->num_samples == 1)
          assert(mt->halign == 16);
-
-      if (brw->gen >= 9) {
-         assert(mt->num_samples > 1 ||
-                brw_losslessly_compressible_format(brw, surf_type));
-      }
-
    }
 
    uint32_t *surf = allocate_surface_state(brw, surf_offset, surf_index);