From a12533f2ce2e5a4aeae0f1fc8d759de73bdb6e2d Mon Sep 17 00:00:00 2001 From: Sagar Ghuge Date: Thu, 15 Nov 2018 16:19:39 -0800 Subject: [PATCH] intel/eu/gen12: Implement immediate 64 bit constant encoding. On Gen12, 64 bit immediate constants are loaded in reverse order. Lower 32 bit gets loaded from bit 96-127 and higher 32 bits from 64-95 in instruction encoding. Signed-off-by: Sagar Ghuge Co-authored-by: Matt Turner Reviewed-by: Francisco Jerez Reviewed-by: Kenneth Graunke Reviewed-by: Jordan Justen --- src/intel/compiler/brw_inst.h | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/src/intel/compiler/brw_inst.h b/src/intel/compiler/brw_inst.h index fdc6cd76225..f5a65fea40d 100644 --- a/src/intel/compiler/brw_inst.h +++ b/src/intel/compiler/brw_inst.h @@ -1136,7 +1136,13 @@ brw_inst_set_imm_df(const struct gen_device_info *devinfo, } dt; (void) devinfo; dt.d = value; - brw_inst_set_bits(insn, 127, 64, dt.u); + + if (devinfo->gen >= 12) { + brw_inst_set_bits(insn, 95, 64, dt.u >> 32); + brw_inst_set_bits(insn, 127, 96, dt.u & 0xFFFFFFFF); + } else { + brw_inst_set_bits(insn, 127, 64, dt.u); + } } static inline void @@ -1144,7 +1150,12 @@ brw_inst_set_imm_uq(const struct gen_device_info *devinfo, brw_inst *insn, uint64_t value) { (void) devinfo; - brw_inst_set_bits(insn, 127, 64, value); + if (devinfo->gen >= 12) { + brw_inst_set_bits(insn, 95, 64, value >> 32); + brw_inst_set_bits(insn, 127, 96, value & 0xFFFFFFFF); + } else { + brw_inst_set_bits(insn, 127, 64, value); + } } /** @} */ -- 2.30.2