On 18 July 2012 15:11, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 07/18/2012 12:16 PM, Paul Berry wrote:<br>
> This patch series cleans up a bug fix I made to Mesa on June 22<br>
> (commit 82d2596: i965: Compute dFdy() correctly for FBOs).  The bug<br>
> was that the i965 driver wasn't adjusting the dFdy() logic to account<br>
> for the fact that window system framebuffers use flipped Y coordinates<br>
> compared to FBOs.  The fix was to compile fragment shaders differently<br>
> when they are used for FBOs vs window system framebuffers.<br>
><br>
> Commit 82d2596 conservatively assumed that all fragment shader<br>
> programs might use dFdy(), so it recompiled all of them when switching<br>
> between FBOs and window system framebuffers.  For shaders that don't<br>
> use dFdy(), the recompile is an unnecessary performance hit.  This<br>
> patch series adds accounting to core mesa to keep track of whether or<br>
> not a shader uses dFdy(), and changes the i965 back-end to only<br>
> recompile shaders that actually need it.<br>
><br>
> Patches 1-3 may be of interest to non-i965 developers, since the<br>
> accounting in core Mesa applies to all back-ends.<br>
><br>
> [PATCH 1/4] mesa: Add UsesDFdy to struct gl_fragment_program.<br>
> [PATCH 2/4] mesa: Set UsesDFdy appropriately for assembly programs.<br>
> [PATCH 3/4] glsl: Set UsesDFdy appropriately for GLSL shaders.<br>
> [PATCH 4/4] i965: Avoid unnecessary recompiles for shaders that don't use dFdy().<br>
<br>
</div></div>Looks fine to me, but please get Ian's ack before pushing.<br></blockquote><div><br>Ian, any comments?<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

<br>
For the series:<br>
Reviewed-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
<br>
<br>
</blockquote></div><br><br>