<div dir="ltr">On 2 December 2013 11:39, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Until now atomic counter built-ins were handled in a way that<br>
prevented the visitor from encountering atomic counter IR variables<br>
and dereferences directly.  In the new surface lowering code it's<br>
going to be more convenient to be able to call back into the visitor<br>
to let it handle the ugly details of atomic counter array<br>
dereferences, and it will make sharing the rest of the atomic<br>
intrinsic handling code easier.<br>
---<br>
 src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 110 +++++++++++++++----------<br>
 1 file changed, 65 insertions(+), 45 deletions(-)<br></blockquote><div><br></div><div>Both of my comments on patch 16 apply here as well.  With those issues addressed, this patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div></div></div></div>