On Thu 02 Feb 2017, Jason Ekstrand wrote: > On Thu, Feb 2, 2017 at 10:13 AM, Chad Versace <[email protected]> > wrote: > > > On Thu 02 Feb 2017, Anuj Phogat wrote: > > > On Thu, Feb 2, 2017 at 9:03 AM, Jason Ekstrand <[email protected]> > > wrote: > > > > On Wed, Feb 1, 2017 at 4:07 PM, Anuj Phogat <[email protected]> > > wrote: > > > >> > > > >> On Wed, Feb 1, 2017 at 2:40 PM, Jason Ekstrand <[email protected]> > > > >> wrote: > > > > > >> > +/** > > > >> > + * Returns true if the given format can support single-sample fast > > > >> > clears. > > > >> > + */ > > > >> > +bool > > > >> > +isl_format_supports_ccs_d(const struct gen_device_info *devinfo, > > > >> > + enum isl_format format) > > > >> > +{ > > > >> > + if (!isl_format_supports_rendering(devinfo, format)) > > > >> > + return false; > > > >> > + > > > >> Add a check for non-MSRT? > > > > I think it should check or assert the gen version, at least. CCS_D first > > appears in gen7. See intel_tiling_supports_non_msrt_mcs(). > > > > agreed. I'll add that.
Then Reviewed-by: Chad Versace <[email protected]> _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
