Merge into main branch rust-lang/rust?

[luojia65]

By now developers have to compile whole custom Rust toolchain to work with xtensa isa. Is there any possibility to merge xtensa instruction set support into main branch Rust? Then developers can install them easily by using rustup toolchain add command.

[mcilloni]

I don't think this can be done until the Xtensa backend is merged into upstream LLVM, but I could be wrong.

[MabezDev]

@mcilloni is correct, Rust won't accept Xtensa support upstream without atleast an experimental backend in LLVM upstream.

With the riscv chips, we've already upstreamed everything we can.

[stappersg]

Where to check the status?

URLs that I'm missing:

• LLVM xtensa branch
• issue that describes the merge of xtensa into LLVM
• issue that describes the merge of "LLVM xtensa" into Rust

[MabezDev]

• https://github.com/espressif/llvm-project
• Tracking Issue for merging to upstream (LLVM-57) espressif/llvm-project#4
• This issue.

[paulocsanz]

espressif/llvm-project#4 implies that all of the repo's changes have been merged to upstream LLVM, although the PR hasn't been closed yet.

What are the next steps?

[igrr]

We have indeed got the first patch set merged into LLVM, however that set of patches is not yet sufficient for a working Rust compiler.
We will update the tracking issue with the next series of patches as soon as we post them for review. You can find the entire set of Xtensa patches here: espressif/llvm-project#62.

[noahbliss]

Hey guys, just wanted to ping this as it seems there is some new traction over at the tracking issue espressif/llvm-project#4 seems this might be coming pretty close

[jessebraham]

seems this might be coming pretty close

My understanding (somebody can correct me if I'm wrong) is that there are still a large number of patches which need to be merged into LLVM before full Xtensa support is realized. Once this is complete, it will take time for these changes to propagate into Rust as well.

[SergioGasquez]

Quick update as we've made some progress about upstreaming our Rust patches, the following PRs were merged:

• Teach rustc about the Xtensa VaListImpl rust-lang/rust#127565
• Add +forced-atomics feature to esp32s2 no_std target rust-lang/rust#133599
• Add std Xtensa targets support rust-lang/rust#126380
• Add no_std Xtensa targets support rust-lang/rust#125141
• Remove - from xtensa targets cpu names rust-lang/rust#138064

We are still awaiting on the LLVM upstreaming effort to be able to continue upstreaming our Rust patches.