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

[wip] Build musl for tests #300

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

Conversation

tgross35
Copy link
Contributor

@tgross35 tgross35 commented Oct 5, 2024

We don't currently test on any non-linux targets because results always require running against the linux-musl target. Also, the current tests don't allow for non-exact comparisons, which would be problematic if we start using implementations with correct rounding.

So: build musl's math functions (can't build the whole library on non-linux systems, but the math functions work) but rename the symbols so we can call them without possibly getting the system libraries.

More experimentation to be done here.

@tgross35 tgross35 force-pushed the test-refactoring branch 3 times, most recently from 073ddcf to 2d8c675 Compare October 6, 2024 03:33
Add a way to run musl symbols

Mostly building

Musl works

Check for undef symbols

Update

Fix error with using relinked archives

More strict archive checks

Clean up warnings

More tests

Disable more warnings

Update build

Update build

Update build

Update syms

Update syms

More tests passing

More tests

Make libm-test a default crate

Remove unneeded files
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

Successfully merging this pull request may close these issues.

1 participant