<div dir="rtl"><div dir="ltr">Hi,</div><div dir="ltr"><br></div><div dir="ltr"><div dir="ltr">Systems without fmaf will very likely miss fmal and all the C99 functions, so they have many other problems to worry about.</div>

<div><br></div><div dir="ltr">LGTM. </div><div dir="ltr"><br></div><div>One question, though. Why</div><div><br></div><div><div> using ::fma;</div><div> using ::fmaf;</div></div><div><br></div><div>but not</div><div><br>
</div>
<div><div> using ::fmal;</div><div><br></div></div><div>This is not fma specific question - cmath repeats this pattern for all C99 functions.</div><div>Shouldn't all three version treated the same?</div><div><br></div>

<div>Yaron</div><div><br></div></div><div dir="ltr"><br></div><div dir="ltr"><br></div><div dir="ltr"><br></div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2014-02-22 17:44 GMT+02:00 Marshall Clow <span dir="ltr"><<a href="mailto:mclow.lists@gmail.com" target="_blank">mclow.lists@gmail.com</a>></span>:</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="http://llvm.org/bugs/show_bug.cgi?id=18910" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=18910</a><br>


<br>
Remove the implementation of "std:::fmaf” from libc++’s <cmath> header, and use the c library’s one instead.<br>
<br>
I believe this bug is a historical relic from the early days of libc++ implementation.<br>
It was part of the original libc++ checkin to the LLVM repo.<br>
Probably very old versions of Apple’s C library were based on C89, which didn’t have fmaf, and so Howard just provided this here.<br>
[ When I say very old, I mean in the Mac OS X 10.3/10.4 timeframe ]<br>
<br>
Now we have C99 libraries in most places, and they all (as far as I can tell) provide fmaf themselves, and (I think) we can remove this.<br>
<br>
If you’re running on a system that has an older C library implementation, this could be a breaking change, and I want to know about it.<br>
I can put this code back under an #ifdef for those systems, but not if I don’t know which systems they are.<br>
<span class="HOEnZb"><font color="#888888"><br>
— Marshall<br>
<br>
</font></span><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>