diff options
| author | Jason Ekstrand <[email protected]> | 2018-04-17 17:12:05 -0700 |
|---|---|---|
| committer | Francisco Jerez <[email protected]> | 2019-10-11 12:24:16 -0700 |
| commit | ca7b6fd3922cd6eb898e9fb2bac90b9933d3a0af (patch) | |
| tree | 8ddf84a62af54d248df42fa594a96459ed13ac14 /src/intel/compiler/brw_eu_validate.c | |
| parent | ab5aa016890c5e7a5dbac2544fd8196fa5a995ed (diff) | |
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 <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/intel/compiler/brw_eu_validate.c')
| -rw-r--r-- | src/intel/compiler/brw_eu_validate.c | 3 |
1 files changed, 2 insertions, 1 deletions
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; } |