Am Freitag, den 22.09.2017, 09:22 +0200 schrieb Samuel Pitoiset: > That's unnecessary to double-check that dcc_offset is not 0 > because all callers already check that.
In that case I would suggest to add an assert(rtex->dcc_offset) to ensure this. Best, Gert > > Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> > --- > src/gallium/drivers/radeonsi/si_blit.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/gallium/drivers/radeonsi/si_blit.c > b/src/gallium/drivers/radeonsi/si_blit.c > index 0ecfc83fe2..2c846d2914 100644 > --- a/src/gallium/drivers/radeonsi/si_blit.c > +++ b/src/gallium/drivers/radeonsi/si_blit.c > @@ -470,7 +470,7 @@ static void si_blit_decompress_color(struct > pipe_context *ctx, > "Decompress Color (levels %u - %u, mask > 0x%x)\n\n", > first_level, last_level, level_mask); > > - if (rtex->dcc_offset && need_dcc_decompress) { > + if (need_dcc_decompress) { > custom_blend = sctx->custom_blend_dcc_decompress; > > /* disable levels without DCC */ _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev