From 0e7bb74a1a6d232139d21cd171c3792180c4bfe4 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Tue, 26 Mar 2024 02:15:47 -0700 Subject: [PATCH] Revert "intel/brw: Don't consider UNIFORM_PULL_CONSTANT_LOAD a send-from-GRF" This reverts commit 5814534de57444475cc3e20c88c4110e7629d9de. It apparently caused GPU hangs in Assassin's Creed: Valhalla, and it isn't that critical of a patch, so let's just roll it back for now. Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/10894 Part-of: --- src/intel/compiler/brw_fs.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index b970312f40c..fb07e10d662 100644 --- a/src/intel/compiler/brw_fs.cpp +++ b/src/intel/compiler/brw_fs.cpp @@ -211,6 +211,8 @@ fs_inst::is_send_from_grf() const case SHADER_OPCODE_MEMORY_FENCE: case SHADER_OPCODE_BARRIER: return true; + case FS_OPCODE_UNIFORM_PULL_CONSTANT_LOAD: + return src[1].file == VGRF; case FS_OPCODE_FB_READ: return src[0].file == VGRF; default: