On Fri, Jul 14, 2017 at 03:26:17PM -0700, Chad Versace wrote: > On Wed 12 Jul 2017, Jason Ekstrand wrote: > > Now that we have an actual aux_usage field, we no longer need the > > complex logic of is_lossless_compressed in order to figure out if a > > miptree is CCS_E compressed. As a side-effect, there is not longer any > > need to overload MSAA_LAYOUT_CMS for CCS_E and we can stop doing so. > > --- > > src/mesa/drivers/dri/i965/brw_blorp.c | 2 +- > > src/mesa/drivers/dri/i965/brw_draw.c | 2 +- > > src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 4 +-- > > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 39 > > ++++-------------------- > > src/mesa/drivers/dri/i965/intel_mipmap_tree.h | 4 --- > > 5 files changed, 10 insertions(+), 41 deletions(-) > > Yes. This patch makes me feel more confident that this miptree code is > correct. > > Reviewed-by: Chad Versace <chadvers...@chromium.org>
I had separate patches for starting to use aux_usage and dropping INTEL_MSAA_LAYOUT_CMS for single sampled. This looks even clearer, thanks: Reviewed-by: Topi Pohjolainen <topi.pohjolai...@intel.com> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
