On Sat, Apr 11, 2015 at 2:13 PM, Matt Turner <matts...@gmail.com> wrote: > Readding mesa-dev@ to Cc. > > On Sat, Apr 11, 2015 at 11:06 AM, Darius Goad <alegen...@gmail.com> wrote: >> Well, I did make a v2, but all that did was fix a bug. How does this not >> come close? > > Well, for instance the i965 driver supports ~5 generations of > hardware, three of them that don't support doubles. There's not a > brw->gen >= ... conditional in any of the patches. > > The third patch modifies ilo, a Gallium driver -- not i965. The > extension isn't actually being enabled on i965. > > I'm not actually convinced the second patch compiles. It uses fabsd(), > which as far as I can tell should just be fabs(). > > Implementing double support is a pretty significant project that's > going to involve some serious understanding of the i965 instruction > set. > > Topi has some work-in-progress branches for fp64 support > (http://cgit.freedesktop.org/~tpohjola/mesa/). Take a look at them to > get a better idea of what's involved. > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Not to mention that this doesn't touch NIR at all, which was just enabled by default for i965 and doesn't support doubles at all ATM. There are also some questions about how that should happen. Connor _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
