I think it's safer to keep this, considering that the call is quite expensive.
Marek On Thu, Jun 15, 2017 at 11:47 AM, Samuel Pitoiset <samuel.pitoi...@gmail.com> wrote: > st/mesa already takes care of this. > > Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> > --- > 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 1cd1f9190e0..ac9358c6b3b 100644 > --- a/src/gallium/drivers/radeonsi/si_state.c > +++ b/src/gallium/drivers/radeonsi/si_state.c > @@ -659,9 +659,6 @@ static void si_set_clip_state(struct pipe_context *ctx, > struct si_context *sctx = (struct si_context *)ctx; > struct pipe_constant_buffer cb; > > - if (memcmp(&sctx->clip_state.state, state, sizeof(*state)) == 0) > - return; > - > sctx->clip_state.state = *state; > si_mark_atom_dirty(sctx, &sctx->clip_state.atom); > > -- > 2.13.1 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev