Skip to content

ci: run stdlib tests for other architectures #2031

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
wants to merge 1 commit into from

Conversation

aykevl
Copy link
Member

@aykevl aykevl commented Aug 6, 2021

Currently this runs the tests for amd64, 386, arm, arm64, and wasi. I
tried adding wasm and a baremetal target (such as riscv-qemu) but they
don't pass the tests yet and should be fixed in the future. At least
this change should make sure that existing tests continue to pass on a
variety of architectures.

Depends on #2030 for 386 and wasm fixes, so marked as draft until then.

@aykevl aykevl force-pushed the test-other-architectures branch from f54ce95 to c798524 Compare August 6, 2021 17:40
@aykevl aykevl mentioned this pull request Aug 7, 2021
@aykevl aykevl force-pushed the test-other-architectures branch from c798524 to 2bf5de7 Compare August 10, 2021 22:58
@aykevl aykevl marked this pull request as ready for review August 10, 2021 23:19
@aykevl
Copy link
Member Author

aykevl commented Aug 10, 2021

This is now ready for review.

@aykevl
Copy link
Member Author

aykevl commented Aug 11, 2021

This fails, rebasing on #2037 (once merged) might fix it.

@deadprogram
Copy link
Member

Ready for rebase @aykevl

@aykevl aykevl force-pushed the test-other-architectures branch from 2bf5de7 to ff8cf71 Compare August 13, 2021 16:26
@aykevl
Copy link
Member Author

aykevl commented Aug 13, 2021

Rebased.

@aykevl
Copy link
Member Author

aykevl commented Aug 15, 2021

Another build failure. Rebasing on #2051 should really fix this - hopefully.

@aykevl aykevl force-pushed the test-other-architectures branch from 929635a to b958e7e Compare November 3, 2021 14:36
@aykevl
Copy link
Member Author

aykevl commented Nov 3, 2021

Rebased. Hopefully all tests pass this time.

Currently this runs the tests for amd64, 386, arm, arm64, and wasi. I
tried adding wasm and a baremetal target (such as riscv-qemu) but they
don't pass the tests yet and should be fixed in the future. At least
this change should make sure that existing tests continue to pass on a
variety of architectures.
@aykevl aykevl force-pushed the test-other-architectures branch from b958e7e to c4fa32e Compare November 3, 2021 14:40
@aykevl
Copy link
Member Author

aykevl commented Nov 3, 2021

Looks like this doesn't quite work yet, the issue appears to be the lack of Go assembly support in TinyGo.

@dkegel-fastly
Copy link
Contributor

See #1906 for the assembly problem.

@deadprogram
Copy link
Member

Closing due to age. Thanks to everyone!

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.

3 participants