<div style="font-family: arial, helvetica, sans-serif"><font size="2"><div class="gmail_quote"><div>+llvm-commits, asl, ddunbar</div><div><br></div><div>Hi, Sebastien!</div><div><br></div><div>The patch doesn't seem that obvious at first glance -</div>
<div>does your assumption about the compiler behavior always hold? (if yes, please add</div><div>an explicit comment about it) Also, I'm not the author of the core compiler-rt lib and</div><div>don't know if optional hardware support is desirable here.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-------- Original Message --------<br>
Subject: Patch: compiler-rt/lib/divsi3.c: do not call udivsi3 directly<br>
Date: Thu, 07 Jun 2012 00:07:10 +0200<br>
From: Sébastien Bourdeauducq <<a href="mailto:sebastien.bourdeauducq@lekernel.net" target="_blank">sebastien.bourdeauducq@<u></u>lekernel.net</a>><br>
To: <a href="mailto:eocallaghan@auroraux.org" target="_blank">eocallaghan@auroraux.org</a>, <a href="mailto:hhinnant@apple.com" target="_blank">hhinnant@apple.com</a>, <a href="mailto:daniel@zuster.org" target="_blank">daniel@zuster.org</a><br>

<br>
Hi,<br>
<br>
The attached patch uses the "/" operator in divsi3 instead of calling<br>
udivsi3 directly. This way, CPUs that have hardware support for unsigned<br>
division do not need to use the slow software emulation. The compiler<br>
will still generate an udivsi3 call for CPUs which lack hardware<br>
unsigned division.<br>
<br>
Please apply.<br>
<br>
Thanks,<br>
Sébastien<br>
<br><br></blockquote></div><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>
</font></div>