-
Notifications
You must be signed in to change notification settings - Fork 266
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
undefined reference to 'frexpl' #502
Comments
By using |
yeah, looks like 8127c9619e2a497a58e6db62df9916e4c4864608 regressed this. new in r16's libandroid_support.a:
removed in r16's libandroid_support.a:
ignoring the
these all seem to just have been an oversight. b04b1a2e39477518e963daaec34fffcb69852c04 claimed we don't need imaxabs/imaxdiv, but we still declare them in the support headers, so i'm not sure that's true.
this is missing from x86 because we have assembler implementations for x86 that come from a different source file than the arm stuff. not sure whether it makes more sense to just -DNDK_SUPPORT_LIBRARY and add an extra #if to the bionic source for this?
these were probably dropped on purpose because they're all before 14, which is the current minimum supported API level. |
suggested fix: https://android-review.googlesource.com/479202 |
Because I want something to copy & paste into the NDK support library test that's slightly better than taking the address of the function... Bug: android/ndk#502 Test: ran tests Change-Id: If43089d16691d6a4dcf5d972450b14ed85bbca81
👍 I'm also seeing linker errors on r16 beta from math when targeting platform 16
|
yeah, log2f was accidentally dropped from libandroid_support.a, which had previously been papering over that crack (log2f wasn't in API 16): https://android-review.googlesource.com/#/c/platform/ndk/+/479202/ |
Dan, I'm still seeing this problem with the ndk 16 beta. Are there steps I should take on my end, or just wait for a future r16 version? sdk/ndk-bundle/sources/cxx-stl/llvm-libc++/include/math.h |
you could try a nightly build until r16beta2: https://android.googlesource.com/platform/ndk/+/master/docs/ContinuousBuilds.md |
frexp/frexpf/frexpl were reported, but imaxabs/imaxdiv were missing too, as was log2f. The nan/nanf/nanl family are also missing, but since they're all available in API level 14 that seems fine. lrintl for x86 remains unresolved by this change. Bug: android/ndk#502 Test: built new tests Change-Id: Id7a9a249e4c7a998fc0570902331d9123bcda7bd
Because I want something to copy & paste into the NDK support library test that's slightly better than taking the address of the function... Bug: android/ndk#502 Test: ran tests Change-Id: If43089d16691d6a4dcf5d972450b14ed85bbca81
Because I want something to copy & paste into the NDK support library test that's slightly better than taking the address of the function... Bug: android/ndk#502 Test: ran tests Change-Id: If43089d16691d6a4dcf5d972450b14ed85bbca81
Description
I've switched from android-ndk-r15-windows-x86_64 to android-ndk-r16-beta1-windows-x86_64.
Linker error:
C:\lib\android\2\android-ndk-r16-beta1-windows-x86_64\android-ndk-r16-beta1\build\tools\stoolchain_x86\bin..\lib\gcc\i686-linux-android\4.9.x........\include\c++\4.9.x/math.h:902: error: undefined reference to 'frexpl'
Environment Details
Standalone toolchain was created by cmd file:
Linker flags:
-shared -Wl,-soname,symbols.so -Wl,--version-script=libcode.version -Wl,--no-undefined -Wl,--fatal-warnings -Wl,--gc-sections -Wl,--icf=safe -llog -lm -landroid -landroid_support -lEGL -lGLESv3 -fPIC -g -DANDROID -ffunction-sections -funwind-tables -fstack-protector-strong -no-canonical-prefixes -mstackrealign -Wa,--noexecstack -Wformat -Werror=format-security -O0 -fno-limit-debug-info -Wl,--build-id -Wl,--warn-shared-textrel -Wl,--fatal-warnings -Wl,--no-undefined -Wl,-z,noexecstack -Qunused-arguments -Wl,-z,relro -Wl,-z,now OBJECTS_FILES_HERE C:\\lib\\android\\2\\android-ndk-r16-beta1-windows-x86_64\\android-ndk-r16-beta1\\build\\tools\\stoolchain_x86\\i686-linux-android\\lib\\libstdc++.a -o libmy.so
The text was updated successfully, but these errors were encountered: