radv: fix pipeline stats mask
Left-shifting by 11*8 or 14*8 is undefined. This fixes many
dEQP-VK.query_pool.statistics_query.* failures (but not pre-existing
flakes) for release builds using clang.
Fixes: 48aabaf225 ("radv: do not harcode the pipeline stats mask for query resolves")
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27651>
This commit is contained in:
@@ -1155,7 +1155,7 @@ radv_query_shader(struct radv_cmd_buffer *cmd_buffer, VkPipeline *pipeline, stru
|
||||
.range = VK_WHOLE_SIZE}}});
|
||||
|
||||
/* Encode the number of elements for easy access by the shader. */
|
||||
pipeline_stats_mask &= (1 << radv_get_pipelinestat_query_size(device)) - 1;
|
||||
pipeline_stats_mask &= (1 << (radv_get_pipelinestat_query_size(device) / 8)) - 1;
|
||||
pipeline_stats_mask |= util_bitcount(pipeline_stats_mask) << 16;
|
||||
|
||||
avail_offset -= src_offset;
|
||||
|
||||
Reference in New Issue
Block a user