<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Aug 25, 2017 at 8:59 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Both are<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>><br>
<br>
(Typo in the title of this patch)<br>
<br>
Should 2/2 go to stable? I'm not really sure how that code gets used.<br>
</blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Probably wouldn't hurt.  Doing a quick grep for the function, I don't see anywhere that it actually matters for those two system values.  We use it in nir_gather_info to fill out system_values_used and in brw_fs_nir because we have a table of pre-computed system values.  However, since they get lowered earlier on, I don't think it matters in practice.<br></div></div>