<div dir="ltr"><div><div>The full commit message is this:<br><br>commit 586b4b500fed64fb724beb3753bc190cd1c676e0<br>Author: Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>><br>Date:   Tue Sep 28 14:32:16 2010 -0700<br>
<br>    glsl: Also update implicit sizes of varyings at link time.<br>    <br>    Otherwise, we'll often end up with gl_TexCoord being 0 length, for<br>    example.  With ir_to_mesa, things ended up working out anyway, as long<br>
    as multiple implicitly-sized arrays weren't involved.<br><br></div>Unfortunately, this commit assumes that vertex shader outputs and fragment shader inputs can be matched up by simply searching for variables with common names in all shaders.  That's not going to be safe in the presence of geometry shaders or interface blocks.<br>
<br></div><div>I tried reverting* the commit and didn't get any piglit failures.  Is it still needed?  I'm having trouble understanding the problem it's trying to solve (in what circumstances would we have ended up with gl_TexCoord being 0 length?)<br>
<br></div><div>*Note: because of merge conflicts I couldn't revert the patch directly, but I did the moral equivalent by restoring the if test in update_array_sizes() to its initial form.  My experiment can be found in branch "revert-update-array-sizes" of git://<a href="http://github.com/stereotype441/mesa.git">github.com/stereotype441/mesa.git</a><br>
</div></div>