As discussed in reply to my series that fixes this for non-SSO paths [1], this series builds on top of that to also handle SSO programs.
I checked that this works by adding SSO ENABLED to these two piglit tests: tests/spec/arb_enhanced_layouts/linker/block-member-locations/named-block-member-location-overlap.shader_test tests/spec/arb_enhanced_layouts/linker/block-member-locations/named-block-member-mixed-order-overlap.shader_test I guess we want to have versions of these with SSO ENABLED in piglit so I'll send patches next week to do that. [1] https://lists.freedesktop.org/archives/mesa-dev/2017-October/173503.html Iago Toral Quiroga (3): glsl/linker: create a helper function to validate explicit locations glsl/linker: generalize validate_explicit_variable_location for SSO glsl/linker: validate explicit locations for SSO programs src/compiler/glsl/link_varyings.cpp | 159 +++++++++++++++++++++++++++--------- src/compiler/glsl/link_varyings.h | 4 + src/compiler/glsl/linker.cpp | 6 ++ 3 files changed, 131 insertions(+), 38 deletions(-) -- 2.11.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
