From ddaa7c4221068bf4679c4b6baaf0da396382635f Mon Sep 17 00:00:00 2001 From: Zach Battleman Date: Wed, 5 Jun 2024 13:23:39 -0700 Subject: [PATCH] intel/brw: update comment to accurately reflect intended behavior Removed mention of Wa_* when referencing an intended harware behavior since version 12. This will prevent the erroneous usage of the `intel_needs_workaround` in the future. Reviewed-by: Mark Janes Part-of: --- src/intel/compiler/brw_fs.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index b75afde6404..45369e53647 100644 --- a/src/intel/compiler/brw_fs.cpp +++ b/src/intel/compiler/brw_fs.cpp @@ -427,7 +427,7 @@ fs_inst::can_do_source_mods(const struct intel_device_info *devinfo) const if (is_send_from_grf()) return false; - /* From Wa_1604601757: + /* From TGL PRM Vol 2a Pg. 1053 and Pg. 1069 MAD and MUL Instructions: * * "When multiplying a DW and any lower precision integer, source modifier * is not supported."