draw: remove clip_vertex from vertex header
authorRoland Scheidegger <sroland@vmware.com>
Fri, 11 Dec 2015 23:50:54 +0000 (00:50 +0100)
committerRoland Scheidegger <sroland@vmware.com>
Tue, 15 Dec 2015 01:03:40 +0000 (02:03 +0100)
commit8e264765a47a4904279b651b688688f5f4d11ea9
tree95c8fbce32f678e595a2fe9114a9ef6864d3909c
parent1775400a20e3b85f83d0d0e99ea7382e965f4b11
draw: remove clip_vertex from vertex header

vertex header had both clip_pos and clip_vertex.
We only really need one (clip_pos) because the draw llvm shader would
overwrite the position output from the vs with the viewport transformed.
However, we don't really need the second one, which was only really used
for gl_ClipVertex - if the shader didn't have that the values were just
duplicated to both clip_pos and clip_vertex. So, just use this from the vs
output instead when we actually need it.
Also change clip debug to output both the data from clip_pos and the
clipVertex output (if available).
Makes some things more complex, some things less complex, but seems more
easy to understand what clipping actually does (and what values it uses
to do its magic).

Reviewed-by: Brian Paul <brianp@vmware.com
Reviewed-by: Jose Fonseca <jfonseca@vmware.com>
src/gallium/auxiliary/draw/draw_cliptest_tmp.h
src/gallium/auxiliary/draw/draw_llvm.c
src/gallium/auxiliary/draw/draw_llvm.h
src/gallium/auxiliary/draw/draw_pipe_clip.c
src/gallium/auxiliary/draw/draw_private.h