diff options
author | Nanley Chery <[email protected]> | 2017-01-21 13:35:50 -0800 |
---|---|---|
committer | Nanley Chery <[email protected]> | 2017-02-03 09:23:13 -0800 |
commit | 84dbf683781241038d847b9db81f153ab59e27f6 (patch) | |
tree | b2c4f77a5dd9bafa662fa56f1c620135f2a0b771 /src/intel/vulkan/anv_blorp.c | |
parent | 93b819154f625c4bf2fe369af35f1d9f93ebf8ce (diff) |
anv/blorp: Disable resolves for transparent black clears
Signed-off-by: Nanley Chery <[email protected]>
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src/intel/vulkan/anv_blorp.c')
-rw-r--r-- | src/intel/vulkan/anv_blorp.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/vulkan/anv_blorp.c index 2cf972bb613..759d2ae9444 100644 --- a/src/intel/vulkan/anv_blorp.c +++ b/src/intel/vulkan/anv_blorp.c @@ -1451,9 +1451,15 @@ ccs_resolve_attachment(struct anv_cmd_buffer *cmd_buffer, resolve_op = BLORP_FAST_CLEAR_OP_RESOLVE_FULL; } else if (att_state->fast_clear) { /* We don't know what to do with clear colors outside the render - * pass. We need a partial resolve. + * pass. We need a partial resolve. Only transparent black is + * built into the surface state object and thus no resolve is + * required for this case. */ - resolve_op = BLORP_FAST_CLEAR_OP_RESOLVE_PARTIAL; + if (att_state->clear_value.color.uint32[0] || + att_state->clear_value.color.uint32[1] || + att_state->clear_value.color.uint32[2] || + att_state->clear_value.color.uint32[3]) + resolve_op = BLORP_FAST_CLEAR_OP_RESOLVE_PARTIAL; } else { /* The image "natively" supports all the compression we care about * and we don't need to resolve at all. If this is the case, we also |