-
-
Notifications
You must be signed in to change notification settings - Fork 3
Support C-SKY (experimental) #32
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
Conversation
58c0f70 to
fb42421
Compare
5e3b045 to
18aadd2
Compare
5b7a8a0 to
d82f7c8
Compare
0e136c1 to
47320db
Compare
2a7e414 to
6f78a13
Compare
I've been a bit busy recently, so I may not have looked into it very thoroughly. However, my initial investigation suggests the compilation error originates from the |
d3fb6a3 to
205bc81
Compare
Thanks for the investigation! I'm not sure why, but quickcheck indeed to be related, disabling quickcheck-related tests allowed the full test suite to build. In multi-threaded tests, hangs or the following assertion failures in glibc or pthread occur (can be reproduced without this crate's atomics). Considering the discussion in rust-lang/rust#117306 that libatomic is broken and likely causing data races, this is an understandable result. As for the issue that QEMU cannot run tests using ldex/stex when non-ck860 CPU is specified (#32 (comment)), looking at the Linux kernel code, only ck860 set CPU_HAS_LDSTEX, so it seems QEMU is correct here.
Updated PR to check the |
be1fabe to
3b18848
Compare
|
As for "multiple definition of `__lll_lock_wait_private'" error, it maybe related to https://sourceware.org/bugzilla/show_bug.cgi?id=15648 that was fixed in glibc 2.34. |
79592ad to
1f75c5f
Compare
This has been resolved by applying a patch to quickcheck that uses fastrand instead of rand. |
Of the CPU architectures supported by Rust, this is the last architectures not supported by this crate.
The build passed, but has not yet been tested. The qemu binary mentioned on the platform support docs does not work with even a minimal program due to the following error:UPDATE: See #32 (comment)TODO: ldex/stex is ck860* only? #32 (comment)UPDATE: See #32 (comment)