[Piglit] [PATCH 0/2] Make gen_vs_in_fp64 even faster

Dylan Baker dylan at pnwbakers.com
Mon Jun 13 18:45:47 UTC 2016


Quoting Andres Gomez (2016-06-13 09:32:17)
> On Fri, 2016-06-10 at 16:42 -0700, Dylan Baker wrote:
> 
> [snip]
> 
> > 
> > I've had this plan to add additional helpers for generator writers
> > for
> > representing GLSL type information, and this seemed like the right
> > time
> > to add them.
> 
> This looks very good. I'm also repeating some value arrays for testing
> each type and, also, several conversion functions from hex to actual
> values so maybe it would be worth to think also about adding this into
> a module.
> 
> [snip]
> 
> > Andres, I saw your series that extends this generator. Do you think
> > it's
> > better to consider landing this first and rebase your series ontop of
> > this, or land yours first and rebase this series?
> 
> Let's land this first and I will rework my series. In any case, I will
> split the biggest patch as for your comments there.
> 
> -- 
> Br,
> 
> Andres
> 
> 

I've changed to use 1 instead of None, and pushed to master. Thanks
again for your review.

Dylan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/piglit/attachments/20160613/55bd05e4/attachment.sig>


More information about the Piglit mailing list