Module: Mesa
Branch: master
Commit: 51376cd74965007b7facf0b86c9d74c678234978
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=51376cd74965007b7facf0b86c9d74c678234978

Author: Jason Ekstrand <jason.ekstr...@intel.com>
Date:   Wed Jun 13 09:23:28 2018 -0700

Revert "intel/compiler: Properly consider UBO loads that cross 32B boundaries."

This reverts commit b8fa847c2ed9c7c743f31e57560a09fae3992f46.

This broke about 30k Vulkan CTS tests.

---

 src/intel/compiler/brw_nir_analyze_ubo_ranges.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/intel/compiler/brw_nir_analyze_ubo_ranges.c 
b/src/intel/compiler/brw_nir_analyze_ubo_ranges.c
index 6d6ccf73ad..d58fe3dd2e 100644
--- a/src/intel/compiler/brw_nir_analyze_ubo_ranges.c
+++ b/src/intel/compiler/brw_nir_analyze_ubo_ranges.c
@@ -141,16 +141,10 @@ analyze_ubos_block(struct ubo_analysis_state *state, 
nir_block *block)
          if (offset >= 64)
             continue;
 
-         /* The value might span multiple 32-byte chunks. */
-         const int bytes = nir_intrinsic_dest_components(intrin) *
-                           (nir_dest_bit_size(intrin->dest) / 8);
-         const int end = DIV_ROUND_UP(offset_const->u32[0] + bytes, 32);
-         const int regs = end - offset + 1;
-
          /* TODO: should we count uses in loops as higher benefit? */
 
          struct ubo_block_info *info = get_block_info(state, block);
-         info->offsets |= ((1ull << regs) - 1) << offset;
+         info->offsets |= 1ull << offset;
          info->uses[offset]++;
       }
    }

_______________________________________________
mesa-commit mailing list
mesa-commit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to