[Mesa-dev] [RFC] More ARB_arrays_of_arrays support

Timothy Arceri t_arceri at yahoo.com.au
Sat Mar 21 23:51:50 PDT 2015


On Sat, 2015-03-21 at 20:06 -0400, Ilia Mirkin wrote:
> Patches 1, 2, 3, 4, 8, and 11 are Reviewed-by: Ilia Mirkin
> <imirkin at alum.mit.edu>
> 
> The rest I sent comments on, although even if you address my feedback,
> I'm not sufficiently familiar to r-b at least some of them.

Thanks for the reviews.

> 
> If you do decide to play with UBO's, you can use idr's excellent
> random_ubo script to stress-test the various cases. Helped me a lot on
> getting doubles to actually work reliably.

Thanks for the tip, I'll take a look at this.

> 
> On Sat, Mar 21, 2015 at 5:49 AM, Timothy Arceri <t_arceri at yahoo.com.au> wrote:
> > This series adds most of the remaining glsl arrays of arrays support. Support for uniform blocks is still missing, I've played around with this but don't have anything working yet.
> >
> > What do you guys think about adding these changes without the uniform blocks support? Adding these changes now I assume would make it easy for the nir guys to start adding AoA support (if it doesn't exist already) and the intel backend changes could start to be worked on.
> >
> > For anyone interested the main remaining changes to be made are in:
> > ./glsl/link_uniform_block_active_visitor.cpp
> > ./glsl/link_uniform_blocks.cpp
> > ./glsl/lower_ubo_reference.cpp
> >
> > For gallium drivers the glsl to tgsi conversion seems to handle everything without changes so at least for the piglit tests I've written everything seem to just work.
> >
> > There are no piglit regressions from these changes (tested on i965 driver) and all the current arrays of arrays piglit test pass (tested with ilo gallium driver using MESA_GL_VERSION_OVERRIDE=3.3COMPAT MESA_GLSL_VERSION_OVERRIDE=150) besides the uniform block tests. There are also some unreviewed interface linking tests relating to the last 4 patches on the piglit list [1]. I also have some generated execution tests in my piglit repo [2], I'm not sure if I've created these the best way and wish to extend them further so these haven't been sent to the piglit list yet.
> >
> > [1] http://lists.freedesktop.org/archives/piglit/2015-March/015167.html
> > [2] https://github.com/tarceri/Mesa_arrays_of_arrays_piglit/tree/execution_tests
> > _______________________________________________
> > mesa-dev mailing list
> > mesa-dev at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/mesa-dev




More information about the mesa-dev mailing list