Skip to content
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

reference: ndk roadmap #409

Closed
headupinclouds opened this issue May 2, 2017 · 8 comments
Closed

reference: ndk roadmap #409

headupinclouds opened this issue May 2, 2017 · 8 comments

Comments

@headupinclouds
Copy link
Collaborator

android/ndk#379 (comment)

libc++ is in much better shape than it was back then, but it's still not quite to the point that I'm comfortable recommending it. For a timeline on that, see our roadmap.

https://android.googlesource.com/platform/ndk/+/master/docs/Roadmap.md

@xsacha
Copy link

xsacha commented May 14, 2018

NDK r17 is officially out with libc++ as default now.

@headupinclouds
Copy link
Collaborator Author

Thanks. I see @ruslo has already added android-ndk-r17-api-24-arm64-v8a-clang-libcxx14 in @polly. We can update the r16b toolchain toolchain to r17.

Caveat: I tested a beta version or r17 and it didn't support MinSizeRel + LTO yet, although a gcc + libc++ combination could be used as a workaround.

@headupinclouds
Copy link
Collaborator Author

testing #656

@xsacha
Copy link

xsacha commented May 14, 2018

You'll need to use CMake 3.12 (unreleased) as well.

@ruslo
Copy link
Collaborator

ruslo commented May 14, 2018

You'll need to use CMake 3.12 (unreleased) as well.

@xsacha CMake 3.11.1 works fine with Android NDK r17:

@xsacha
Copy link

xsacha commented May 14, 2018

Oh right. Some of my packages complain about FindThreads in NDK r17. Thought it would affect this project too.

@xsacha
Copy link

xsacha commented May 14, 2018

Also, Boost will fail to compile with NDK r17 if you target API 21,22,23. Only works on older APIs or 24 and above due to removal of libandroid_support.
See: android/ndk#480 (comment)

@headupinclouds
Copy link
Collaborator Author

Merged #656 ... Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants