OpenHarmony/third_party_mesa3d
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

intel/eu/gen12: Add sanity-check asserts to brw_inst_bits() and brw_inst_set_bits().

Browse Source 
These caught a few bugs during the development of this series.

Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
Francisco Jerez
2018-11-09 14:13:35 -08:00
parent 7e5a8638d3
commit 143176163d
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/intel/compiler/brw_inst.h
View File

@@ -1161,6 +1161,7 @@ BRW_IA16_ADDR_IMM(send_dst, -1, -1, 62, 56, 52)
static inline uint64_t static inline uint64_t
brw_inst_bits(const brw_inst *inst, unsigned high, unsigned low) brw_inst_bits(const brw_inst *inst, unsigned high, unsigned low)
{ {
assert(high >= low);
/* We assume the field doesn't cross 64-bit boundaries. */ /* We assume the field doesn't cross 64-bit boundaries. */
const unsigned word = high / 64; const unsigned word = high / 64;
assert(word == low / 64); assert(word == low / 64);
@@ -1181,6 +1182,7 @@ brw_inst_bits(const brw_inst *inst, unsigned high, unsigned low)
static inline void static inline void
brw_inst_set_bits(brw_inst *inst, unsigned high, unsigned low, uint64_t value) brw_inst_set_bits(brw_inst *inst, unsigned high, unsigned low, uint64_t value)
{ {
assert(high >= low);
const unsigned word = high / 64; const unsigned word = high / 64;
assert(word == low / 64); assert(word == low / 64);