Skip to content

Verify libc builds with --cfg stdbuild #68

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

Closed
nagisa opened this issue Nov 21, 2015 · 6 comments
Closed

Verify libc builds with --cfg stdbuild #68

nagisa opened this issue Nov 21, 2015 · 6 comments

Comments

@nagisa
Copy link
Member

nagisa commented Nov 21, 2015

Android CI only check whether tests can be run and pass, but do not check whether library actually builds under Android.

This is important to test, because e.g. librt does not exist in Android devkit (and liblibc does not actually use it either) and we still try to link to it since #66.

@alexcrichton
Copy link
Member

I don't understand what's at play here, if we're ensuring tests run and pass (which requires the library to build) then I don't see what the "flaw" is.

This library doesn't actually link anything unless passed --cfg stdbuild, which may be what you're referring to, but it's not clear to me.

@nagisa
Copy link
Member Author

nagisa commented Nov 22, 2015

The test binary itself is built outside of the simulator, which means that we do not check whether test binary actually builds under android. Only that it runs and all tests pass.

@alexcrichton
Copy link
Member

To the best of my knowledge there's no compiler in the simulator itself, the only option for compiling is outside. Even if there were, however, there's no Rust compiler that runs in the simulator and there's no reason cross compiling should be a problem.

Can you elaborate a bit more on why you opened this? Was it to detect how -lrt is missing on Android?

@nagisa
Copy link
Member Author

nagisa commented Nov 22, 2015

Can you elaborate a bit more on why you opened this? Was it to detect how -lrt is missing on Android?

Exactly :)

@alexcrichton alexcrichton changed the title Android CI is flawed Verify libc builds with --cfg stdbuild Nov 22, 2015
@alexcrichton
Copy link
Member

Ok, in that case I've updated the title to reflect what's going on. There's no CI that this library builds with --cfg stdbuild, and there's also no verification that it links with that. Android is thoroughly tested right now, so that's not the problem, it's moreso that nothing tests the linkage of this library because that's what the standard library is responsible for.

One half of this will be building with --cfg stdbuild (on nightly only), and the second half will be figuring out how to link that rlib into a binary to test the linkage. I don't know how to do the second half.

@alexcrichton
Copy link
Member

This is now basically done by #146 as we build with no_std

danielverkamp pushed a commit to danielverkamp/libc that referenced this issue Apr 28, 2020
* SSSE3: _mm_abs_epi16, _mm_abs_epi32, _mm_hadd_epi16

* SSSE3: _mm_hadds_epi16

* SSSE3: assert_instr

* SSSE3: _mm_hadd_epi32

* SSSE3: _mm_hsub_epi16

* SSSE3: _mm_hsubs_epi16

* SSSE3: _mm_hsub_epi32

* SSSE3: _mm_maddubs_epi16

* SSSE3: _mm_mulhrs_epi16

* SSSE3: _mm_sign_epi8

* SSSE3: _mm_sign_epi32

* SSSE3: _mm_sign_epi32

* SSSE3: Fix assert_instr
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants