Skip to content
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

LLVM warnings emitted for Xtensa target-cpu #137150

Open
Noratrieb opened this issue Feb 16, 2025 · 1 comment
Open

LLVM warnings emitted for Xtensa target-cpu #137150

Noratrieb opened this issue Feb 16, 2025 · 1 comment
Assignees
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. C-bug Category: This is a bug. O-xtensa T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@Noratrieb
Copy link
Member

I ran rustc --target $TARGET for

  • xtensa-esp32-espidf
  • xtensa-esp32-none-elf
  • xtensa-esp32s2-espidf
  • xtensa-esp32s2-none-elf
  • xtensa-esp32s3-espidf
  • xtensa-esp32s3-none-elf

And for all of them, I got a warning that 'esp32' is not a recognized processor for this target (ignoring processor) (or esp32-s2 or esp32-s3 respectively).

Additionally for xtensa-esp32s2-none-elf, I also got '+forced-atomics' is not a recognized feature for this target (ignoring feature)

@MabezDev @SergioGasquez as Xtensa maintainers

@Noratrieb Noratrieb added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. C-bug Category: This is a bug. O-xtensa T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Feb 16, 2025
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Feb 16, 2025
@Noratrieb Noratrieb removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Feb 16, 2025
@MabezDev
Copy link
Contributor

Hi @Noratrieb, this intended, at least for a while.

LLVM patches are still being upstreamed and once they land, they take a while to propagate to the LLVM that rustc uses. We've applied these patches already to ease the upstreaming effort later, but also to reduce the patch set I have to maintain on our out of tree fork: https://github.com/esp-rs/rust which uses the full LLVM backend.

We can keep this open to track this, feel free to assign myself and @SergioGasquez

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-target-feature Area: Enabling/disabling target features like AVX, Neon, etc. C-bug Category: This is a bug. O-xtensa T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

3 participants