Use bit-wise not instead of logical not.
authorIan Romanick <ian.d.romanick@intel.com>
Thu, 18 Mar 2010 22:35:05 +0000 (15:35 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Thu, 18 Mar 2010 22:35:05 +0000 (15:35 -0700)
The assertion is checking that the low-order bits of offset are not
set.  It does this by anding the inverted offset mask with the
offset.  This is clearly intended to be a bit-wise "invert".

Fixes bug #25984.

src/mesa/drivers/dri/i915/intel_tris.c

index fb191fe346f16e3842324adb94b3585e7268d833..4b6d3b4c5be54cabf08f6b8cb6705b6527e62980 100644 (file)
@@ -251,7 +251,7 @@ void intel_flush_prim(struct intel_context *intel)
       BEGIN_BATCH(5);
       OUT_BATCH(_3DSTATE_LOAD_STATE_IMMEDIATE_1 |
                I1_LOAD_S(0) | I1_LOAD_S(1) | 1);
-      assert((offset & !S0_VB_OFFSET_MASK) == 0);
+      assert((offset & ~S0_VB_OFFSET_MASK) == 0);
       OUT_RELOC(vb_bo, I915_GEM_DOMAIN_VERTEX, 0, offset);
       OUT_BATCH((intel->vertex_size << S1_VERTEX_WIDTH_SHIFT) |
                (intel->vertex_size << S1_VERTEX_PITCH_SHIFT));