[Mesa-dev] [PATCH 0/3] Enable ARB_gpu_shader_fp64 for i965

Connor Abbott cwabbott0 at gmail.com
Sat Apr 11 14:19:00 PDT 2015


On Sat, Apr 11, 2015 at 2:13 PM, Matt Turner <mattst88 at gmail.com> wrote:
> Readding mesa-dev@ to Cc.
>
> On Sat, Apr 11, 2015 at 11:06 AM, Darius Goad <alegend45 at 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 at 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


More information about the mesa-dev mailing list