i965/gen6: Fix assign instead of compare in assert
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 30 Sep 2011 20:48:18 +0000 (21:48 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 30 Sep 2011 20:49:29 +0000 (21:49 +0100)
This is from a Coverity defect report.

src/mesa/drivers/dri/i965/brw_vec4_emit.cpp
   268  static void
   269  check_gen6_math_src_arg(struct brw_reg src)
   270  {
   271     /* Source swizzles are ignored. */
   272     assert(!src.abs);
   273     assert(!src.negate);
-> 274     assert(src.dw1.bits.swizzle = BRW_SWIZZLE_XYZW);
   275  }

Reported-by: Vinson Lee <vlee@vmware.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=40214
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
src/mesa/drivers/dri/i965/brw_vec4_emit.cpp

index 1eb8f3f78a34e59726ded8a4194bbb621c20698c..3feecbf805de32f64831d78afb54ed29a43158ec 100644 (file)
@@ -257,7 +257,7 @@ check_gen6_math_src_arg(struct brw_reg src)
    /* Source swizzles are ignored. */
    assert(!src.abs);
    assert(!src.negate);
-   assert(src.dw1.bits.swizzle = BRW_SWIZZLE_XYZW);
+   assert(src.dw1.bits.swizzle == BRW_SWIZZLE_XYZW);
 }
 
 void