Re: [Mesa-dev] [PATCH 5/5] draw: remove clip_vertex from vertex header

2015-12-14 Thread Jose Fonseca
On 12/12/15 02:31, srol...@vmware.com wrote: From: Roland Scheidegger 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

Re: [Mesa-dev] [PATCH 5/5] draw: remove clip_vertex from vertex header

2015-12-14 Thread Brian Paul
The series looks good to me. Thanks for digging into this. Reviewed-by: Brian Paul On 12/11/2015 07:31 PM, srol...@vmware.com wrote: From: Roland Scheidegger vertex header had both clip_pos and clip_vertex. We only really need one (clip_pos) because the draw llvm shader would overwrite th

[Mesa-dev] [PATCH 5/5] draw: remove clip_vertex from vertex header

2015-12-11 Thread sroland
From: Roland Scheidegger 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_