[PATCH] D22244: Android: Conditionally enable long-double math.h inline overloads

Dan Albert via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 12 18:29:12 PDT 2016


danalbert added a comment.

https://android.googlesource.com/platform/development/+/master/ndk/platforms/android-18/include/math.h#410

The current bionic header gives us a clearer picture: https://android.googlesource.com/platform/bionic/+/master/libc/include/math.h#240

So yes, most of these were missing until somewhat recently. `libandroid_support` provides these things though (if it's missing any, that's still the right place to fix this).

We've also debated just defaulting to using a static libm for Android since all that stuff can be safely statically linked across platform versions, so there's really no reason to not always use the latest version.


http://reviews.llvm.org/D22244





More information about the llvm-commits mailing list