diff options
author | Jason Ekstrand <[email protected]> | 2018-06-13 09:23:28 -0700 |
---|---|---|
committer | Jason Ekstrand <[email protected]> | 2018-06-13 09:23:55 -0700 |
commit | 51376cd74965007b7facf0b86c9d74c678234978 (patch) | |
tree | c2deb32be2db26d227c32025fb7920c53d2b74eb | |
parent | b8fa847c2ed9c7c743f31e57560a09fae3992f46 (diff) |
Revert "intel/compiler: Properly consider UBO loads that cross 32B boundaries."
This reverts commit b8fa847c2ed9c7c743f31e57560a09fae3992f46.
This broke about 30k Vulkan CTS tests.
-rw-r--r-- | src/intel/compiler/brw_nir_analyze_ubo_ranges.c | 8 |
1 files changed, 1 insertions, 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 6d6ccf73ade..d58fe3dd2e3 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]++; } } |