intel/eu: Use GET_BITS in brw_inst_set_send_ex_desc
authorJason Ekstrand <jason.ekstrand@intel.com>
Thu, 15 Nov 2018 21:05:57 +0000 (15:05 -0600)
committerJason Ekstrand <jason@jlekstrand.net>
Tue, 29 Jan 2019 18:43:55 +0000 (18:43 +0000)
It's a bit more readable

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
src/intel/compiler/brw_inst.h

index ce89bbba72f3a90a832aa75c5259fec1340418ae..753f09e70cfc9d541f6f4bdd70ba9881d28a4ac8 100644 (file)
@@ -513,11 +513,11 @@ brw_inst_set_send_ex_desc(const struct gen_device_info *devinfo,
                           brw_inst *inst, uint32_t value)
 {
    assert(devinfo->gen >= 9);
-   brw_inst_set_bits(inst, 94, 91, (value >> 28) & ((1u << 4) - 1));
-   brw_inst_set_bits(inst, 88, 85, (value >> 24) & ((1u << 4) - 1));
-   brw_inst_set_bits(inst, 83, 80, (value >> 20) & ((1u << 4) - 1));
-   brw_inst_set_bits(inst, 67, 64, (value >> 16) & ((1u << 4) - 1));
-   assert((value & ((1u << 16) - 1)) == 0);
+   brw_inst_set_bits(inst, 94, 91, GET_BITS(value, 31, 28));
+   brw_inst_set_bits(inst, 88, 85, GET_BITS(value, 27, 24));
+   brw_inst_set_bits(inst, 83, 80, GET_BITS(value, 23, 20));
+   brw_inst_set_bits(inst, 67, 64, GET_BITS(value, 19, 16));
+   assert(GET_BITS(value, 15, 0) == 0);
 }
 
 /**