On Fri, 2 Sep 2011 21:21:10 -0700, Paul Berry <stereotype...@gmail.com> wrote: Non-text part: multipart/alternative > On 2 September 2011 18:37, Eric Anholt <e...@anholt.net> wrote: > > > This series gets gen4 to be non-regressing for the new vertex shader. > > I'd be fine with not pushing the last patch and letting Paul's patches > > land, then fixing the bug as it remains there. Once these two land, I > > think it's time to turn on the new backend by default. > > > > This plan sounds good to me. I'd like to land my patches as soon as I can, > since my clip distance work depends on them. Based on your review, I think > only two issues remain with my patch series:
The main issue I saw was "where do those two result helper functions live". I'd be fine with leaving them in the driver if it means the code lands sooner. 4a727e79621b52a6dfcd5c9d5e7a7c6567a422f6 and 29738bda57660b92cc55ae59fdf0259a3b9c7bc4 get my review.
pgpKYxarRVAGK.pgp
Description: PGP signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
