<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, May 20, 2018 at 2:21 PM, Mathias Fröhlich <span dir="ltr"><<a href="mailto:Mathias.Froehlich@gmx.net" target="_blank">Mathias.Froehlich@gmx.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Marek,<br>
<span><br>
On Sunday, 20 May 2018 20:08:08 CEST Marek Olšák wrote:<br>
> The old code saves which vertex element is the first to use a VBO slot.<br>
> When VBOs are added to the buffer list, each VBO is added only for such<br>
> vertex elements, and not added for others. So the old and new code do<br>
> exactly the same thing but differently.<br>
<br>
</span>But its doing that with less calls to radeon_add_to_buffer_list/amdg<wbr>pu_cs_add_buffer.<br>
It avoids duplicate calls for pipe_vertex_elements refering to the same pipe_vertex_buffer.<br></blockquote><div><br></div><div>And the old code does exactly the same thing - it avoids duplicate calls for the same pipe_vertex_buffer, because vertex elements reusing vertex buffer slots don't have their bit set in first_vb_use_mask, so add_to_buffer_list is never called for those vertex elements.</div><div><br></div><div>Marek<br></div></div></div></div>