diff options
author | Mike Blumenkrantz <michael.blumenkrantz@gmail.com> | 2021-03-30 16:22:27 -0400 |
---|---|---|
committer | Marge Bot <eric+marge@anholt.net> | 2021-06-02 13:31:43 +0000 |
commit | 79742d41c00b70cc751d1d83a9c83a0a93f9ef01 (patch) | |
tree | f9033db2f8664dfed55b1fe7954630931834b708 | |
parent | 4fd9c6d9f35046d65c1751678e72a538dce72774 (diff) |
radv: declare index_va in a single call for indexed draw packet emit
this is an extreme hotpath, so having a single calculation in a const
variable is slightly better for compiler microoptimizing
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11124>
-rw-r--r-- | src/amd/vulkan/radv_cmd_buffer.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/amd/vulkan/radv_cmd_buffer.c b/src/amd/vulkan/radv_cmd_buffer.c index dc6ac362665..45dcd281c1b 100644 --- a/src/amd/vulkan/radv_cmd_buffer.c +++ b/src/amd/vulkan/radv_cmd_buffer.c @@ -5354,7 +5354,6 @@ radv_emit_draw_packets_indexed(struct radv_cmd_buffer *cmd_buffer, { const struct radv_cmd_state *state = &cmd_buffer->state; const int index_size = radv_get_vgt_index_size(state->index_type); - uint64_t index_va; uint32_t remaining_indexes = cmd_buffer->state.max_index_count; remaining_indexes = MAX2(remaining_indexes, info->first_index) - info->first_index; @@ -5364,8 +5363,7 @@ radv_emit_draw_packets_indexed(struct radv_cmd_buffer *cmd_buffer, cmd_buffer->device->physical_device->rad_info.has_zero_index_buffer_bug) return; - index_va = state->index_va; - index_va += first_index * index_size; + const uint64_t index_va = state->index_va + first_index * index_size; if (!state->subpass->view_mask) { radv_cs_emit_draw_indexed_packet(cmd_buffer, index_va, remaining_indexes, count); |