From: Jason Ekstrand Date: Wed, 18 Apr 2018 00:12:05 +0000 (-0700) Subject: intel/eu/validate/gen12: Don't blow up on indirect src0. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=ca7b6fd3922cd6eb898e9fb2bac90b9933d3a0af;p=mesa.git intel/eu/validate/gen12: Don't blow up on indirect src0. They look like a NULL source if you don't look at the address mode. Reviewed-by: Francisco Jerez Reviewed-by: Kenneth Graunke --- diff --git a/src/intel/compiler/brw_eu_validate.c b/src/intel/compiler/brw_eu_validate.c index 5e76087a922..9e9033910b1 100644 --- a/src/intel/compiler/brw_eu_validate.c +++ b/src/intel/compiler/brw_eu_validate.c @@ -170,7 +170,8 @@ dst_is_null(const struct gen_device_info *devinfo, const brw_inst *inst) static bool src0_is_null(const struct gen_device_info *devinfo, const brw_inst *inst) { - return brw_inst_src0_reg_file(devinfo, inst) == BRW_ARCHITECTURE_REGISTER_FILE && + return brw_inst_src0_address_mode(devinfo, inst) == BRW_ADDRESS_DIRECT && + brw_inst_src0_reg_file(devinfo, inst) == BRW_ARCHITECTURE_REGISTER_FILE && brw_inst_src0_da_reg_nr(devinfo, inst) == BRW_ARF_NULL; }