Jordan's gen6-layer work made me try if some of the duplicated switch cases could be shared. Not a big deal, but a little clearer I think.
There is still one left in gen7_blorp_emit_depth_stencil_config() but there depth is resolved based on miptree alone instead of consulting a renderbuffer. I suspect we should be able to use the miptree as the source in all cases but didn't look into it in detail. Topi Pohjolainen (3): i965/gen7: Refactor renderbuffer target override i965/gen7: Refactor array type resolving i965/gen7: Refactor resolving of depth src/mesa/drivers/dri/i965/brw_state.h | 18 ++++++++++++ src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 32 +++++++++++++++++++++ src/mesa/drivers/dri/i965/gen7_blorp.cpp | 4 +-- src/mesa/drivers/dri/i965/gen7_misc_state.c | 34 +++-------------------- src/mesa/drivers/dri/i965/gen7_wm_surface_state.c | 28 +++---------------- 5 files changed, 59 insertions(+), 57 deletions(-) -- 1.8.3.1 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
