[Mesa-dev] [00/21] Last of SSO, take 47

[Ian Romanick]

On 04/30/2014 12:41 PM, Eric Anholt wrote:
> Ian Romanick <idr at freedesktop.org> writes:
> 
>> A lot the patches in this series were slightly reworked to incorporate
>> Eric's feedback (remove ir_variable::user_location) on the previous
>> attempt.  Other than that change, I patch 1 is new, and patch 16 adds
>> previously missing display list support.
> 
> Patch 2, 3, 13, 16, 17, 18, 20, 21 are:
> 
> Reviewed-by: Eric Anholt <eric at anholt.net>
> 
> I think my patch 15 comments were minor, and if you agree with them
> (meaning I actually understood what was going on), then apply the r-b
> there too.
> 
> Patch 19 is:
> 
> Acked-by: Eric Anholt <eric at anholt.net>
> 
> Patch 14 I don't think is needed (since we've got explicit locations
> being assigned to corresponding varying slots).

The sorting is so that a vertex shader that does

    out vec4 a;
    out vec4 b;

and a fragment shader that does

    in vec4 b;
    in vec4 a;

will assign the same locations for a and b.  If both shaders have the
same set of varyings, they'll all get the same locations.  Without
sorting, we assign the locations based on the order in which variables
appear in the shader text.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140430/1b1e107e/attachment.sig>