gallium/u_threaded: do not apply start twice
We already apply start when setting the bits in writable_buffer, so
shifting the bits up by start just makes us apply the offset twice.
Caught by Coverity.
Fixes: 988d091720 ("gallium/u_threaded: clear valid buffer range only if it's not bound for write")
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11635>
This commit is contained in:
committed by
Marge Bot
parent
28e1b02a6f
commit
669b6889e4
@@ -1467,7 +1467,7 @@ tc_set_shader_images(struct pipe_context *_pipe,
|
||||
}
|
||||
|
||||
tc->image_buffers_writeable_mask[shader] &= ~BITFIELD_RANGE(start, count);
|
||||
tc->image_buffers_writeable_mask[shader] |= writable_buffers << start;
|
||||
tc->image_buffers_writeable_mask[shader] |= writable_buffers;
|
||||
}
|
||||
|
||||
struct tc_shader_buffers {
|
||||
|
||||
Reference in New Issue
Block a user