<p><br>
Am 29.09.2017 14:51 schrieb "Marek Olšák" <<a href="mailto:maraeo@gmail.com">maraeo@gmail.com</a>>:</p>
<p>><br>
> If all requirements are met, UARL isn't emitted and the source operand<br>
> of UARL is folded into the instruction where ADDR would normally be<br>
> used.<br>
 I only skimmed over the patches, but this will need tracking  reladdr* in the temporary register  lifetime estimation that is called by merge_registers, which is not yet done (unless I missed a patch in the last few days or I didn't see it in this set).</p>
<p>AFAIR radeonsi doesn't use it, but at least r600g does, for which you also enabled this new behaviour.</p>
<p>On Monday I can have a closer look at it.</p>
<p>best,<br>
gert<br><br></p>
<p>><br>
> The use case that we primarily care about is to get SV[i].x in address<br>
> operands to help us generate better code in radeonsi.<br>
><br>
> Please review.<br>
><br>
> Thanks,<br>
> Marek<br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>