On 09/26/2017 12:45 PM, Nicolai Hähnle wrote:
Gert's suggestion makes sense. Apart from that, the series is
Yeah, I will add an assert over there, thanks!
Reviewed-by: Nicolai Hähnle <[email protected]>
On 22.09.2017 09:22, Samuel Pitoiset wrote:
That's unnecessary to double-check that dcc_offset is not 0
because all callers already check that.
Signed-off-by: Samuel Pitoiset <[email protected]>
---
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
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
