-
Notifications
You must be signed in to change notification settings - Fork 99
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
musl libc-test report 77 bugs #236
Comments
Thanks for the report! I think there's always room for a bit more CI in a repo like this. Could you elaborate a bit on the bugs you're finding? Are these bugs that need fixing in the source code? Or are these sort of like 'interface bugs' where this library doesn't do things like set errno? |
Sorry I did not link the gist with the filtered report : The original report is 6k lines of error mostly due to not setting errno and failing on all the test related to long double functions. After filtering those out it still reports 77 bugs that may need fixing in the source code. The first bug on acosh
The third bug for j0
|
Ah ok, makes sense! I think it'd be great to fix these accuracy bugs for sure, and if we could include the test script that'd be great too! For the errno stuff I think we'll want to basically ignore that for now, I don't think this library will want to explore an ABI-compatible solution in that respect. |
I have pushed every thing in PR #237. What I have in mind is a process like this:
If every one is ok with that I don't mind submitting a PR adding the 77 failing test as uni-test. As for fixing the accuracy bugs, I'm not sure where to start my current plan is to build two version of the same software, one using musl, the other one using rust-libm and debug them step by step. |
As an exercise, I have exposed rust-lang/libm as a c library and linked it against musl test suite libc-test
It founded 77 bugs excluding all bug related to wrong error handling and long double.
The bugs are located in : acosh, asinh, j0, jn, jnf, lgamma, sinh, tgamma, y0, y0f, ynf.
All the code can be found on this branch.
You can run the test using the test-musl.sh script.
I'm not sure how to proceed. Should we expose a c abi library and linking it with other libm test suite or port all the test as rust unit test ?
The text was updated successfully, but these errors were encountered: