<div dir="ltr">On 21 August 2013 23:26, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">We're soon going to be calling brw_alloc_reg_set() from outside of the<br>
visitor, where we don't have the precomputed "max_grf" variable handy.<br>
<br>
Signed-off-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
Cc: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br></blockquote><div><br></div><div>Thanks for doing this, Ken!  Series is:<br></div><div><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
<br></div><div>Possible follow-up idea: maybe we should move max_grf out of fs_visitor and vec4_visitor, and instead put it into brw and initialize it in brwCreateContext(), like we do for brw->max_wm_threads etc.<br></div>
</div></div></div>