[Piglit] [PATCH 2/2] arb_gpu_shader_int64: add builtin test generator.

Dylan Baker dylan at pnwbakers.com
Fri Jun 10 16:35:30 UTC 2016


Quoting Dave Airlie (2016-06-09 22:27:40)
> On 10 June 2016 at 13:40, Dave Airlie <airlied at gmail.com> wrote:
> >> No, you can't use int and long here, python 3 doesn't have a long type
> >> at all, and it's int will grow until it uses all of your memory if you
> >> ask it to, while python2 has an int and long that are equivalent to int
> >> and long in C for your architecture. You'll need to use the numpy types
> >> exclusively. As-is this doesn't work with python 3.x it raises a
> >> NameError, so that at least needs to be fixed.
> >>
> >> Also, some of the helper functions in here seem either useless, wrong,
> >> or at least need to have their documentation updated. Many of of them
> >> use floats, which will cause your numpy.integer types to get converted
> >> to their numpy.float equivalent, which you probably don't want, unless
> >> the code casts back, this code is pretty complicated.
> >>
> >> There's also a heck of a lot of this code that is duplicated or nearly
> >> duplicated between builtin_function.py, builtin_function_fp64.py and
> >> this, although I don't have a good solution and I don't really want to
> >> to de-duplicate all of that code myself, so I guess you can just ignore
> >> this comment, consider it me complaining to no one in particular.
> >
> > You made me feel guilty, so I added support to the main generator script,
> > it seems to work fine.
> >
> > patch on the list.
> 
> btw I tried to reintegrate the fp64 generator and failed miserably, at some
> point it started generating two versions of the same test.
> 
> Dave.

I was actually thinking about having a look at the generators and seeing
what we could do about making integrating things like half-float and
[u]int{8,16} easier. I thought there were extensions that exposed types
smaller than 32bit, so it's maybe worth having support for them in the
generators.

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/20160610/71eb02f4/attachment.sig>


More information about the Piglit mailing list