<div dir="ltr">Hi Zack,<div><br></div><div>This change regresses a bunch of point sprite piglit tests on i915g. Should we revert back to the old behaviour? As far as I can see, it was correct (it was keeping the attributes in case another stage is using them).</div>

<div><br></div><div>Stéphane</div><div><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 8, 2013 at 12:46 PM, Zack Rusin <span dir="ltr"><<a href="mailto:zackr@vmware.com" target="_blank">zackr@vmware.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Before inserting new front face and prim id outputs cleanup<br>
the old extra outputs, otherwise our cache will use previous<br>
output slots which will break as soon as outputs of the current<br>
shader don't match the last.<br>
<br>
Signed-off-by: Zack Rusin <<a href="mailto:zackr@vmware.com">zackr@vmware.com</a>><br>
---<br>
 src/gallium/auxiliary/draw/draw_context.c |    1 +<br>
 1 file changed, 1 insertion(+)<br>
<br>
diff --git a/src/gallium/auxiliary/draw/draw_context.c b/src/gallium/auxiliary/draw/draw_context.c<br>
index af9caee..2dc6772 100644<br>
--- a/src/gallium/auxiliary/draw/draw_context.c<br>
+++ b/src/gallium/auxiliary/draw/draw_context.c<br>
@@ -555,6 +555,7 @@ draw_get_shader_info(const struct draw_context *draw)<br>
 void<br>
 draw_prepare_shader_outputs(struct draw_context *draw)<br>
 {<br>
+   draw_remove_extra_vertex_attribs(draw);<br>
    draw_ia_prepare_outputs(draw, draw->pipeline.ia);<br>
    draw_unfilled_prepare_outputs(draw, draw->pipeline.unfilled);<br>
 }<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.10.4<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div></div>