Skip to content

[Hotfix] Exchange rust-xtensa repository. - #1

Closed
0ndorio wants to merge 1 commit into
ctron:masterfrom
0ndorio:fix/rustc_with_compiler_builtins_v0.1.18
Closed

[Hotfix] Exchange rust-xtensa repository.#1
0ndorio wants to merge 1 commit into
ctron:masterfrom
0ndorio:fix/rustc_with_compiler_builtins_v0.1.18

Conversation

@0ndorio

@0ndorio 0ndorio commented Jul 25, 2019

Copy link
Copy Markdown
Contributor

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

0ndorio commented Jul 25, 2019

Copy link
Copy Markdown
Contributor Author

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