<div dir="ltr">Hi Quentin,<div><br></div><div>I committed this patch as r208934, and will analyze those regression with your patch.</div><div><br></div><div>Thanks,</div><div>-Jiangning</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">2014-05-16 0:57 GMT+08:00 Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Jiangning,<br>
<div class=""><br>
> If I don't have other higher priority tasks, I will have a look. But it would be much better if your patch of fixing MOVAddr issue can be shared with me.<br>
<br>
</div>Sure! It is more a hack than a patch, but here it is :).<br>
{F57887}<br>
<div class=""><br>
> Is there any possibility after this optimization the live range of base address is extended, and triggered spill/fill in hot loop by RA? If this is the case the rematerialization needs to be tuned, I think.<br>
<br>
</div>I thought about that and this is possible. Though the current implementation still uses the pseudo instructions so the rematerialization should kick in. That said, this is also possible that the rematerialization algorithm is not perfect.<br>

<div class=""><br>
> Actually I was hoping your patch can get it enabled at least for ARM64. :-)<br>
<br>
</div>For ARM64, yes, but for ARM64-apple-* no :). We need to address the regressions first before enabling it by default for our (apple) triple, but if you need it for other OS, I am fine with that, though I cannot test the performances for those.<br>

We can land the patch with it disabled by default for all targets then enabled it for the targets (triple) we care along the way.<br>
<br>
Thanks for checking the alignment thing.<br>
<br>
Yes, I am happy with that :).<br>
<br>
Thanks,<br>
-Quentin<br>
<br>
<a href="http://reviews.llvm.org/D3432" target="_blank">http://reviews.llvm.org/D3432</a><br>
<br>
<br>
</blockquote></div><br></div>