-
Notifications
You must be signed in to change notification settings - Fork 17
Support thumb-mode Armv7-R, Armv7-A and Armv8-R #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
base: main
Are you sure you want to change the base?
Conversation
24e6f70 to
96f06f3
Compare
|
In order to be able to test thumb-mode Armv7-R and Armv7-A, we need to be able to mark some functions as being compiled in Thumb mode. However, the armv7a, armv7r and armv8r targets didn't have the thumb-interworking option set. They now do, but only in the latest nightly, not the last stable. I'm wondering in the QEMU tests should always just use nightly and not bother with stable? |
|
Or I guess I could write the function in assembly code... |
96f06f3 to
7ef1f3a
Compare
|
Rebased so it uses #100, and modified to run QEMU tests on nightly (although the libraries themselves are still build on stable and our MSRV). |
|
My justfile changes aren't right - this should fail the build due to missing targets upstream and it does not. |
f02a244 to
7f0a85b
Compare
7f0a85b to
0d4402c
Compare
Also relies on all the targets having thumb-interworking enabled.
Thanks, macOS :/
You can't depend on the flags because they vary depending on what the machine code does.
0d4402c to
4dad2b2
Compare
7013484 to
15d2330
Compare
Switching from sh to bash might make the `if` work. In my defense, it worked fine on macOS...
They were created before I rebased on main, which had changes to the example programs.
Now fixed. And that then meant I had to update the outputs. |
Also relies on all the targets having thumb-interworking enabled.
Merge this one before #98