diff options
| author | Marek Olšák <[email protected]> | 2016-04-17 15:34:24 +0200 |
|---|---|---|
| committer | Marek Olšák <[email protected]> | 2016-04-18 19:51:25 +0200 |
| commit | 3faecdd4e13a773e188d7b1c7647e61ee0aa50bb (patch) | |
| tree | 23578398f939d1f7b2e20714a1b4d63a049aa92b | |
| parent | 28c2573b4f1b311145b3f21a1794adb3dcd5f61a (diff) | |
radeonsi: remove TODO and correct a comment in si_emit_cache_flush
Yes, that flag is really needed.
Reviewed-by: Nicolai Hähnle <[email protected]>
| -rw-r--r-- | src/gallium/drivers/radeonsi/si_state_draw.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index 40cad504e09..8f4eba4151e 100644 --- a/src/gallium/drivers/radeonsi/si_state_draw.c +++ b/src/gallium/drivers/radeonsi/si_state_draw.c @@ -615,7 +615,7 @@ void si_emit_cache_flush(struct si_context *si_ctx, struct r600_atom *atom) * doesn't seem to work reliably. Since the bug doesn't affect * correctness (it only does more work than necessary) and * the performance impact is likely negligible, there is no plan - * to fix it. + * to add a workaround for it. */ if (sctx->flags & SI_CONTEXT_INV_ICACHE) @@ -628,7 +628,6 @@ void si_emit_cache_flush(struct si_context *si_ctx, struct r600_atom *atom) if (sctx->flags & SI_CONTEXT_INV_GLOBAL_L2) { cp_coher_cntl |= S_0085F0_TC_ACTION_ENA(1); - /* TODO: this might not be needed. */ if (sctx->chip_class >= VI) cp_coher_cntl |= S_0301F0_TC_WB_ACTION_ENA(1); } |