intel/brw: Avoid copy propagating any fixed registers into EOTs

We were handling FIXED_GRF, but we probably also ought to handle ATTR
(pushed inputs) and UNIFORM (pushed constants).  Just check if file
isn't VGRF to handle everything.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27876>
This commit is contained in:
Kenneth Graunke 2024-01-03 02:58:54 -08:00 committed by Marge Bot
parent 97bf3d3b2d
commit 49606ab067

View file

@ -689,8 +689,8 @@ try_copy_propagate(const brw_compiler *compiler, fs_inst *inst,
* anything that would make it impossible to satisfy that restriction.
*/
if (inst->eot) {
/* Avoid propagating a FIXED_GRF register, as that's already pinned. */
if (entry->src.file == FIXED_GRF)
/* Don't propagate things that are already pinned. */
if (entry->src.file != VGRF)
return false;
/* We might be propagating from a large register, while the SEND only