diff options
| author | Francisco Jerez <[email protected]> | 2018-11-09 14:13:35 -0800 |
|---|---|---|
| committer | Francisco Jerez <[email protected]> | 2019-10-11 12:24:16 -0700 |
| commit | 143176163d39720fa040b441be6290286590e5cf (patch) | |
| tree | 98b6e7242cd6623c0478041d4843e8b2c4466a75 /src/intel/compiler/brw_inst.h | |
| parent | 7e5a8638d3a2a3b0fdd71be196ce241714827dd6 (diff) | |
intel/eu/gen12: Add sanity-check asserts to brw_inst_bits() and brw_inst_set_bits().
These caught a few bugs during the development of this series.
Reviewed-by: Caio Marcelo de Oliveira Filho <[email protected]>
Reviewed-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/intel/compiler/brw_inst.h')
| -rw-r--r-- | src/intel/compiler/brw_inst.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/intel/compiler/brw_inst.h b/src/intel/compiler/brw_inst.h index 624cbe020ca..5a1e5cfba72 100644 --- a/src/intel/compiler/brw_inst.h +++ b/src/intel/compiler/brw_inst.h @@ -1161,6 +1161,7 @@ BRW_IA16_ADDR_IMM(send_dst, -1, -1, 62, 56, 52) static inline uint64_t brw_inst_bits(const brw_inst *inst, unsigned high, unsigned low) { + assert(high >= low); /* We assume the field doesn't cross 64-bit boundaries. */ const unsigned word = high / 64; assert(word == low / 64); @@ -1181,6 +1182,7 @@ brw_inst_bits(const brw_inst *inst, unsigned high, unsigned low) static inline void brw_inst_set_bits(brw_inst *inst, unsigned high, unsigned low, uint64_t value) { + assert(high >= low); const unsigned word = high / 64; assert(word == low / 64); |