From ca7b6fd3922cd6eb898e9fb2bac90b9933d3a0af Mon Sep 17 00:00:00 2001 From: Jason Ekstrand Date: Tue, 17 Apr 2018 17:12:05 -0700 Subject: [PATCH] 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 --- src/intel/compiler/brw_eu_validate.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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; } -- 2.30.2