<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Apr 1, 2017 9:25 AM, "Joerg Sonnenberger" <<a href="mailto:joerg@bec.de">joerg@bec.de</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="quoted-text">On Fri, Mar 31, 2017 at 08:10:33PM -0700, Davide Italiano wrote:<br>
> On Fri, Mar 31, 2017 at 12:58 PM, Joerg Sonnenberger via llvm-commits<br>
> <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br>
> > Author: joerg<br>
> > Date: Fri Mar 31 14:58:07 2017<br>
> > New Revision: 299247<br>
> ><br>
> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=299247&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=299247&view=rev</a><br>
> > Log:<br>
> > Do not translate rint into nearbyint, but truncate it like nearbyint.<br>
> ><br>
><br>
> I think Eli pointed out this already in the past, but aren't<br>
> llvm.rint()/llvm.nearbyint() equal and should we remove rint until we<br>
> have a better story for FP exception modeling?<br>
<br>
</div>Please see the discussion in phab: they are not equal as one is allowed<br>
to raise exceptions and the other not. In practise this means that at<br>
least software implementations of nearbyint tend to wrap rint with<br>
exception glue. That's the original motivation for the change -- it<br>
breaks BSD libm builds and pessimizes code in general.<br>
<font color="#888888"><br>
Joerg<br>
</font></blockquote></div><br></div></div><div class="gmail_extra" dir="auto">Definitely haven't seen that, now it makes sense. Please add a phab link to the bottom of your commit message to make sure the review is visible and/or add such an explanation to the message itself next time.</div><div class="gmail_extra" dir="auto"><br></div><div class="gmail_extra" dir="auto">--</div><div class="gmail_extra" dir="auto">Davide</div></div>