[PATCH] D39611: [CodeGen] make cbrt and fma constant (never set errno); document complex calls as always constant

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 3 13:35:08 PDT 2017


efriedma added inline comments.


================
Comment at: include/clang/Basic/Builtins.def:169
+// Disregard that 'cbrt' could set errno with a NaN input. The C standard says
+// that NaN arguments generally do not raise FP exceptions.
+BUILTIN(__builtin_cbrt , "dd", "Fnc")
----------------
Not sure adding this comment specifically for cbrt adds anything; the same could be said of a bunch of other functions, like ceil().


https://reviews.llvm.org/D39611





More information about the cfe-commits mailing list