Module: Mesa
Branch: main
Commit: 3c83ac800212f9347c8550fe1938a8b7a5617e83
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=3c83ac800212f9347c8550fe1938a8b7a5617e83

Author: Marcin Ĺšlusarz <[email protected]>
Date:   Thu Jul 20 19:42:12 2023 +0200

intel/compiler: remove redundant code

has_lsc is checked few lines above, so this code doesn't matter.

Added in: a358b97c586 ("intel/fs: optimize uniform SSBO & shared loads")

Reviewed-by: Sagar Ghuge <[email protected]>
Reviewed-by: Lionel Landwerlin <[email protected]>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24260>

---

 src/intel/compiler/brw_nir_blockify_uniform_loads.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/src/intel/compiler/brw_nir_blockify_uniform_loads.c 
b/src/intel/compiler/brw_nir_blockify_uniform_loads.c
index d28d6a4adf6..d5cbfbc9072 100644
--- a/src/intel/compiler/brw_nir_blockify_uniform_loads.c
+++ b/src/intel/compiler/brw_nir_blockify_uniform_loads.c
@@ -78,12 +78,6 @@ brw_nir_blockify_uniform_loads_instr(nir_builder *b,
       if (nir_dest_bit_size(intrin->dest) != 32)
          return false;
 
-      /* Without the LSC, we can only do block loads of at least 4dwords (1
-       * oword).
-       */
-      if (!devinfo->has_lsc && nir_dest_num_components(intrin->dest) < 4)
-         return false;
-
       intrin->intrinsic = nir_intrinsic_load_shared_uniform_block_intel;
       return true;
 

Reply via email to