i965: fix up asserts in brw_inst_set_jip()
authorTimothy Arceri <t_arceri@yahoo.com.au>
Thu, 26 Jan 2017 02:50:41 +0000 (13:50 +1100)
committerTimothy Arceri <t_arceri@yahoo.com.au>
Sun, 29 Jan 2017 23:17:46 +0000 (10:17 +1100)
We are casting from a signed 32bit int to an unsigned 16bit int
so shift 15 bits rather than 16.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/brw_inst.h

index 13fce97e5398436c3493949ae94338144a84f3d1..bcb6786bbadc8208e47532e2608f53a214a2079c 100644 (file)
@@ -283,8 +283,8 @@ brw_inst_set_jip(const struct gen_device_info *devinfo,
    if (devinfo->gen >= 8) {
       brw_inst_set_bits(inst, 127, 96, (uint32_t)value);
    } else {
-      assert(value <= (1 << 16) - 1);
-      assert(value > -(1 << 16));
+      assert(value <= (1 << 15) - 1);
+      assert(value >= -(1 << 15));
       brw_inst_set_bits(inst, 111, 96, (uint16_t)value);
    }
 }