<div dir="ltr">Thanks for looking into the issue. I'll keep an eye on the bots and let you know if the issue is fixed.<div><br></div><div>Cheers,</div><div>Arnaud</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 15, 2016 at 8:23 PM, Tim Northover via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Arnaud,<br>
<span class=""><br>
> On 15 Apr 2016, at 06:27, Arnaud De Grandmaison <<a href="mailto:Arnaud.DeGrandmaison@arm.com">Arnaud.DeGrandmaison@arm.com</a>> wrote:<br>
> Would you mind having a look at those ?<br>
<br>
</span>Sure. It looks like hoisting constants out of div/rem instructions (particularly in loops) prevents them being converted into cheaper multiplies. This only causes a slowdown for hardware I have access to on bigfib, but I see the same pattern in fasta's assembly.<br>
<br>
Long term, it's probably another issue for GlobalISel. For now, it doesn't seem horribly unreasonable to special-case it in the cost model. I've committed r266464 which should fix it.<br>
<br>
Thanks for bisecting the issue.<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>