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

Louis Dionne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 10:16:30 PDT 2022


ldionne added a comment.

This has to be coordinated with the stack starting at D135777 <https://reviews.llvm.org/D135777> or this will be hell to rebase, and/or duplicate work's going to happen.



================
Comment at: libcxx/include/math.h:794
+inline _LIBCPP_HIDE_FROM_ABI float acos(float __x) _NOEXCEPT {
+#      if __has_builtin(__builtin_acosf)
+  return __builtin_acosf(__x);
----------------
Can't we assume that we always have the builtin available?


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

https://reviews.llvm.org/D134938



More information about the llvm-commits mailing list