Skip to content

[Hotfix] Exchange rust-xtensa repository. #1

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

Conversation

0ndorio
Copy link
Contributor

@0ndorio 0ndorio commented Jul 25, 2019

Right now it is not possible to compile libcore with the xtensa fork of the rust compiler (rev bba6c06 esp-rs/rust#2).

The reasons are some flaws inside the calculation of required registers to pass function arguments. These got triggered through a recent update of the compiler_builtins helper crate which is dynamically injected into the build process of libcore via crates.io.

As long as this is the case we must are required to use this fork which contains a bunch of fixes.

A pull request to review & merge these fixes is already open (esp-rs/rust#3).


Btw. thanks for your great work to make this toolchain more usable :)

Right now it is not possible to compile libcore with the xtensa
fork of the rust compiler (rev `bba6c06`).

The reasons are some flaws inside the calculation of required
registers for argument passing. These bugs got triggered through
a recent update of the `compiler_builtins` helper crate which is
dynamically injected into the build process of libcore via crates.io.

As long as this is the case we are required to use a fork which
contains related fixes.
@0ndorio
Copy link
Contributor Author

0ndorio commented Jul 25, 2019

No longer required as esp-rs/rust#3 got merged.

@0ndorio 0ndorio closed this Jul 25, 2019
@0ndorio 0ndorio deleted the fix/rustc_with_compiler_builtins_v0.1.18 branch August 20, 2019 17:36
@0ndorio 0ndorio restored the fix/rustc_with_compiler_builtins_v0.1.18 branch September 19, 2019 10:56
@0ndorio 0ndorio deleted the fix/rustc_with_compiler_builtins_v0.1.18 branch September 19, 2019 10:57
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