[Mesa-dev] [PATCH v4 11/18] i965/blorp: Add aux_buf variable to simplify code.
Rafael Antognolli
rafael.antognolli at intel.com
Thu Mar 8 16:49:04 UTC 2018
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.antognolli at 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
More information about the mesa-dev
mailing list