Initial Xtensa support #481
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Please add this architecture to the |
|
Thanks for the link to #430, was most helpful! I've gone ahead and implemented the relevant relocations for Xtensa and added the Xtensa arch to the test case as you requested. |
philipc
requested changes
Nov 8, 2022
MabezDev
force-pushed
the
feature/xtensa
branch
3 times, most recently
from
dd7903d
to
c2a7303
Compare
* Add reloc constants to elf.rs * Implement elf read/write for relocations * Add Xtens arch to test elf_any test case
MabezDev
force-pushed
the
feature/xtensa
branch
from
c2a7303
to
9edd035
Compare
|
Thanks, @philipc! Are there any plans to cut a new release anytime soon? |
|
I'd like to do a release soon. However, there's a bit of XCOFF work in progress currently that I'd like to see polished a bit more first. I may consider doing a release with the XCOFF feature disabled somehow. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the bare minimum Xtensa support to allow our Xtensa-enabled Rust fork to compile on stable 1.65, as we were running into rust-lang/rust#102290 in our testing.