In a follow up patch, we make use of clear_color_bo, which is in mt->mcs_buf or mt->hiz_buf. To avoid duplicating more code that does the same thing on both aux buffers, just use aux_buf already.
Signed-off-by: Rafael Antognolli <rafael.antogno...@intel.com> --- src/mesa/drivers/dri/i965/brw_blorp.c | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index 1d586e5ef38..ffd957fb866 100644 --- a/src/mesa/drivers/dri/i965/brw_blorp.c +++ b/src/mesa/drivers/dri/i965/brw_blorp.c @@ -155,10 +155,13 @@ blorp_surf_for_miptree(struct brw_context *brw, }; struct isl_surf *aux_surf = NULL; + struct intel_miptree_aux_buffer *aux_buf = NULL; if (mt->mcs_buf) - aux_surf = &mt->mcs_buf->surf; + aux_buf = mt->mcs_buf; else if (mt->hiz_buf) - aux_surf = &mt->hiz_buf->surf; + aux_buf = mt->hiz_buf; + + aux_surf = &aux_buf->surf; if (mt->format == MESA_FORMAT_S_UINT8 && is_render_target && devinfo->gen <= 7) @@ -180,16 +183,8 @@ blorp_surf_for_miptree(struct brw_context *brw, .mocs = surf->addr.mocs, }; - if (mt->mcs_buf) { - surf->aux_addr.buffer = mt->mcs_buf->bo; - surf->aux_addr.offset = mt->mcs_buf->offset; - } else { - assert(mt->hiz_buf); - assert(surf->aux_usage == ISL_AUX_USAGE_HIZ); - - surf->aux_addr.buffer = mt->hiz_buf->bo; - surf->aux_addr.offset = mt->hiz_buf->offset; - } + surf->aux_addr.buffer = aux_buf->bo; + surf->aux_addr.offset = aux_buf->offset; } else { surf->aux_addr = (struct blorp_address) { .buffer = NULL, -- 2.14.3 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev