-
Notifications
You must be signed in to change notification settings - Fork 360
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
Complete basic support for android #3608
Comments
maybe, like it s been done with illumos, we might not need to bother with memalign but using posix_memalign instead (android api 16/17 min. I believe). |
Good point. Our target page says
I don't know what concretely that means for our current minimum API level. The current LTS version seems to be r26d, and according to this the minimum API level supported there is 21 -- so if posix_memalign is indeed in 16 or 17 we should be able to use it. @chriswailes do you think we can switch the Rust allocator to |
I opened a PR to use posix_memalign on Android: rust-lang/rust#125244 |
make basic things work on Android Fixes #3608
make basic things work on Android Fixes rust-lang/miri#3608
The goal is to get this command to pass:
I think all that's missing is
libc::memalign
The text was updated successfully, but these errors were encountered: