Skip to content

Android NDK r28b #41293

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

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Android NDK r28b #41293

wants to merge 22 commits into from

Conversation

dg0yt
Copy link
Contributor

@dg0yt dg0yt commented Oct 2, 2024

NDK r27b still produces some misscompiles, cf. #40339.
https://github.com/android/ndk/releases/tag/r28-beta1

NDK r28 problems:

NDK r28

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 7, 2024

Success with "r29-canary" from android/ndk#2081 (comment):

Success Fail Cascade
arm-neon-android 1592 21 26
arm64-android 1639 22 27
x64-android 1666 22 27

@dg0yt
Copy link
Contributor Author

dg0yt commented Nov 22, 2024

Success with "r28-beta2":

Success Fail Cascade
arm-neon-android 1616 13 19
arm64-android 1674 12 19
x64-android 1666 17 33

@dg0yt
Copy link
Contributor Author

dg0yt commented Jan 19, 2025

hdf5: ld.lld: error: /<...>/libdl.a(libdl_static.o): Invalid record,
android/ndk#1591 (comment)

@dg0yt
Copy link
Contributor Author

dg0yt commented Apr 22, 2025

Note to self: created local snapshot android-r28-v0.

@dg0yt
Copy link
Contributor Author

dg0yt commented Jun 5, 2025

The remaining regressions are due to poor code uncovered by deliberate LLVM libc++ changes and also affect new XCode environments.

cpprestsdk:

implicit instantiation of undefined template 'std::char_traits<unsigned char>'
in instantiation of template class 'std::basic_string<unsigned char>

No fix. "cpprestsdk is in maintenance mode."

mygui:

implicit instantiation of undefined template 'std::char_traits<unsigned short>'
in instantiation of template class 'std::basic_string<unsigned short>'

MyGUI/mygui@0731785

ogre-next

implicit instantiation of undefined template 'std::char_traits<unsigned short>'
in instantiation of template class 'std::basic_string<unsigned short>'

Check again with 3.0.0 (from Oct 2024)

paho-mqtt:

implicit instantiation of undefined template 'std::char_traits<unsigned char>'
in instantiation of template class 'std::basic_string<unsigned char>'

No fix.

poppler:

implicit instantiation of undefined template 'std::char_traits<unsigned short>'
in instantiation of template class 'std::basic_string<unsigned short>'

Retry with updated version. (Needs C++20. Related: #42851)

sleepy-discord:

implicit instantiation of undefined template 'std::char_traits<unsigned char>'
in instantiation of template class 'std::basic_string<unsigned char>

No fix.

telnetpp:

implicit instantiation of undefined template 'std::char_traits<unsigned char>'
in instantiation of template class 'std::basic_string<unsigned char>'

No fix.

@dg0yt dg0yt changed the title Test with android NDK r28 beta Android NDK r28b Jun 5, 2025
@dg0yt dg0yt marked this pull request as ready for review June 6, 2025 05:58
@BillyONeal BillyONeal added the depends:vm-update PR contains changes to the VM provisioning scripts label Jun 11, 2025
@BillyONeal
Copy link
Member

I see, so the same stuff that blocked updating macOS block current Android too :(

@dg0yt
Copy link
Contributor Author

dg0yt commented Jun 13, 2025

... to the point where I see it as non-blockers. If llvm libc++ is right.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depends:vm-update PR contains changes to the VM provisioning scripts requires:testing Needs tests added before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants