[Mesa-dev] [PATCH 00/50] GL_ARB_gpu_shader_int64... this time for sure!

[Matt Turner]

On 12/05, Matt Turner wrote:
>On 11/28, Ian Romanick wrote:
>>From: Ian Romanick <ian.d.romanick at intel.com>
>>    Patches 42 through 50 enable the extension on BDW+.
>
>42-48 are
>
>Reviewed-by: Matt Turner <mattst88 at gmail.com>
>
>I don't understand the 64-bit CMP issue, so I'm booting a SKL to see how
>fp64 works.

Ah, I think  I see. Because 16x doubles take up 4 registers, we have to
emit two CMP instructions, one with 1Q and one with 2Q:

cmp.ge.f0(8)    null<1>DF       g2.2<0,1,0>DF   (abs)g11<4,4,1>DF { align1 1Q };
cmp.ge.f0(8)    null<1>DF       g2.2<0,1,0>DF   (abs)g7<4,4,1>DF { align1 2Q };

(from fs-op-add-double-double.shader_test)

Makes sense to me. 49 is

Reviewed-by: Matt Turner <mattst88 at gmail.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Digital signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161205/e6766bf4/attachment-0001.sig>