[PATCH] D134938: Use builtins in <math.h>

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 14 09:47:16 PDT 2022


hubert.reinterpretcast added inline comments.


================
Comment at: libcxx/include/math.h:800-802
+inline _LIBCPP_HIDE_FROM_ABI long double acos(long double __x) _NOEXCEPT {
+#      if __has_builtin(__builtin_acosl) && !defined(_AIX)
+  return __builtin_acosl(__x);
----------------
hubert.reinterpretcast wrote:
> LLVM on AIX currently only supports the case where `double` and `long double` have the same format; that may explain why there is no `__builtin_acosl`?
Hmm. I was only speculating that there was no `__builtin_acosl`, but it seems that is not the case.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134938/new/

https://reviews.llvm.org/D134938



More information about the llvm-commits mailing list