On 05/26/2017 04:42 PM, Marek Olšák wrote:
On Fri, May 26, 2017 at 1:45 AM, Samuel Pitoiset
<[email protected]> wrote:
err, used in si_begin_new_cs(). Ignore for now.
Why? The patch seems good. It's not very important because radeonsi is
behind u_threaded_context. Anyway:
Because I needed some sleep. :)
Reviewed-by: Marek Olšák <[email protected]>
Marek
On 05/26/2017 01:42 AM, Samuel Pitoiset wrote:
cso_set_blend_color() already checks if the old state is different.
Only Nine uses pipe::set_blend_color() directly but I guess it
should use the cache too.
Signed-off-by: Samuel Pitoiset <[email protected]>
---
src/gallium/drivers/radeonsi/si_state.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index 363f32170a..ae7c91f475 100644
--- a/src/gallium/drivers/radeonsi/si_state.c
+++ b/src/gallium/drivers/radeonsi/si_state.c
@@ -619,9 +619,6 @@ static void si_set_blend_color(struct pipe_context
*ctx,
{
struct si_context *sctx = (struct si_context *)ctx;
- if (memcmp(&sctx->blend_color.state, state, sizeof(*state)) == 0)
- return;
-
sctx->blend_color.state = *state;
si_mark_atom_dirty(sctx, &sctx->blend_color.atom);
}
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
