i965: Fix assignment instead of comparison in asserts.
authorVinson Lee <vlee@freedesktop.org>
Sat, 26 Jan 2013 07:27:50 +0000 (08:27 +0100)
committerChad Versace <chad.versace@linux.intel.com>
Mon, 28 Jan 2013 21:51:10 +0000 (13:51 -0800)
Fixes side effect in assertion defects reported by Coverity.

Note: This is a candidate for the 9.1 branch.
Signed-off-by: Vinson Lee <vlee@freedesktop.org>
Reviewed-by: Chad Versace <chad.versace@linux.intel.com>
src/mesa/drivers/dri/i965/brw_fs_emit.cpp

index d9ed27c..45072da 100644 (file)
@@ -951,8 +951,8 @@ fs_generator::generate_pack_half_2x16_split(fs_inst *inst,
 {
    assert(intel->gen >= 7);
    assert(dst.type == BRW_REGISTER_TYPE_UD);
-   assert(x.type = BRW_REGISTER_TYPE_F);
-   assert(y.type = BRW_REGISTER_TYPE_F);
+   assert(x.type == BRW_REGISTER_TYPE_F);
+   assert(y.type == BRW_REGISTER_TYPE_F);
 
    /* From the Ivybridge PRM, Vol4, Part3, Section 6.27 f32to16:
     *