intel: Use bit-wise not instead of logical not (i830 path)
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 19 Mar 2010 00:30:15 +0000 (17:30 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Fri, 19 Mar 2010 00:30:15 +0000 (17:30 -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 4b6d3b4c5be54cabf08f6b8cb6705b6527e62980..81c4adeaf34e44b2675107e3496fd77093863180 100644 (file)
@@ -270,7 +270,7 @@ void intel_flush_prim(struct intel_context *intel)
       OUT_BATCH(_3DSTATE_LOAD_STATE_IMMEDIATE_1 |
                I1_LOAD_S(0) | I1_LOAD_S(2) | 1);
       /* S0 */
-      assert((offset & !S0_VB_OFFSET_MASK_830) == 0);
+      assert((offset & ~S0_VB_OFFSET_MASK_830) == 0);
       OUT_RELOC(vb_bo, I915_GEM_DOMAIN_VERTEX, 0,
                offset | (intel->vertex_size << S0_VB_PITCH_SHIFT_830) |
                S0_VB_ENABLE_830);