[Mesa-dev] [PATCH v2 000/103] i965 Haswell ARB_gpu_shader_fp64 / OpenGL 4.0

Samuel Iglesias Gonsálvez siglesias at igalia.com
Tue Dec 13 11:45:28 UTC 2016


On Tue, 2016-12-13 at 09:01 +0100, Samuel Iglesias Gonsálvez wrote:
> 
[...]
> > i965/vec4/nir: implement double comparisons
> > 
> > 	Trivial: A newline before the if() would be nice.
> > 
> > 	I have a memory of Curro telling me that the hardware maps each
> > 	32-bit chunk in the dst to a single bit in the flag register.
> > 	Maybe that's only on IVB, and maybe I'm misremembering. I'm
> > 	concerned that while the PICK_LOW+MOV will properly handle the
> > 	result that is written to the destination, the result written
> > to
> > 	the flag register might be incorrect.
> > 
> > 	My commit d9b09f8a30 fixed some problems that seems similar in
> > 	my mind.
> > 
> 
> As far as we know that is not what happens, and the flag register has
> one bit for each logical channel (so each 64-bit chunk for DF
> instructions). If that were not the case, I'd expect a lot of the
> tests
> for doubles to fail or at least non-uniform control-flow scenarios to
> fail, for which we have specific tests that are passing just fine in
> both haswell and ivybridge. We will try to double-check with Curro
> just
> in case though.
> 

We have just found an old email from Curro saying that it works as we
think (one bit per logical channel). Maybe Curro wants to confirm it (I
added him on Cc).

Sam
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161213/67a6ba09/attachment-0001.sig>


More information about the mesa-dev mailing list