Browse Source

i965/clear: Remove an early return in fast_clear_depth

Reduce complexity and allow the next patch to delete some code. With
this change, clear operations will still be skipped and setting the
aux_state will cause no side-effects.

Remove the associated comment which implies an early return.

Reviewed-by: Rafael Antognolli <rafael.antognolli@intel.com>
Reviewed-by: Jason Ekstrand <jason@jlekstrand.net>
tags/18.2-branchpoint
Nanley Chery 7 years ago
parent
commit
5b315f3ad1
1 changed files with 0 additions and 5 deletions
  1. 0
    5
      src/mesa/drivers/dri/i965/brw_clear.c

+ 0
- 5
src/mesa/drivers/dri/i965/brw_clear.c View File

@@ -231,10 +231,6 @@ brw_fast_clear_depth(struct gl_context *ctx)
}

if (!need_clear) {
/* If all of the layers we intend to clear are already in the clear
* state then simply updating the miptree fast clear value is sufficient
* to change their clear value.
*/
if (devinfo->gen >= 10 && !same_clear_value) {
/* Before gen10, it was enough to just update the clear value in the
* miptree. But on gen10+, we let blorp update the clear value state
@@ -255,7 +251,6 @@ brw_fast_clear_depth(struct gl_context *ctx)
}
brw_emit_pipe_control_flush(brw, PIPE_CONTROL_STATE_CACHE_INVALIDATE);
}
return true;
}

for (unsigned a = 0; a < num_layers; a++) {

Loading…
Cancel
Save