From: Mathias Fröhlich <mathias.froehl...@gmx.net>

Hi,

Thanks for the review!

This is the starting point to a patch series that I intent to feed.
The aim is to get rid of some VERT_ATTRIB_MAX long loops that currently
happen at about any draw call.

The series tries to separate out one of the depencies of the attribute
aliasing code as a preparation of the above. It uses static const tables
for the aliasing mapping. Also there is some cleanup to make use of
these tables.

v2: Use GLubyte instead of unsigned char
    Remove _ from enum *_MAX values.
    Change comment style.
    Add Comments.
    Put 2 empty lines between functions.
    Change log message.

This series was now rebased to v1 several times during the past weeks.
Nevertheless the original v1 series past the rebases did not regress
piglit quick on radeonsi, i965 and classical swrast. The v2 version
incorporating the reiview comments was not yet run on piglit.
Please review.

thanks and best
Mathias

Mathias Fröhlich (8):
  vbo: Correctly handle attribute offsets in dlist draw.
  mesa: Use defines for the aliased material array attributes.
  mesa: Put materials at the end of the generic block.
  mesa: Track position/generic0 aliasing in the VAO.
  vbo: Use static const VERT_ATTRIB->VBO_ATTRIB maps.
  vbo: Simplify input array distribution for array type draws.
  vbo: Simplify input array distribution for imm type draws.
  vbo: Simplify input array distribution for dlist type draws.

 src/compiler/shader_enums.h                     |  15 +++
 src/mesa/drivers/dri/nouveau/nouveau_render_t.c |   2 +-
 src/mesa/drivers/dri/nouveau/nouveau_vbo_t.c    |   8 +-
 src/mesa/drivers/dri/nouveau/nv10_render.c      |  10 +-
 src/mesa/drivers/dri/nouveau/nv20_render.c      |  20 ++--
 src/mesa/main/arrayobj.c                        | 131 ++++++++++++++++++++++++
 src/mesa/main/arrayobj.h                        |  74 +++++++++++++
 src/mesa/main/enable.c                          |   5 +
 src/mesa/main/ffvertex_prog.c                   |   7 +-
 src/mesa/main/mtypes.h                          |  18 ++++
 src/mesa/main/varray.c                          |  18 +++-
 src/mesa/tnl/t_context.h                        |   6 +-
 src/mesa/vbo/vbo_context.c                      |  23 +----
 src/mesa/vbo/vbo_exec.c                         |  74 +++++++++++++
 src/mesa/vbo/vbo_exec_array.c                   | 115 ++++++---------------
 src/mesa/vbo/vbo_exec_draw.c                    |  81 +++------------
 src/mesa/vbo/vbo_private.h                      |  16 ++-
 src/mesa/vbo/vbo_save.h                         |   1 +
 src/mesa/vbo/vbo_save_api.c                     |  19 ++++
 src/mesa/vbo/vbo_save_draw.c                    |  86 ++++------------
 20 files changed, 457 insertions(+), 272 deletions(-)

-- 
2.14.3
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to