forked from rust-lang/rust
-
Notifications
You must be signed in to change notification settings - Fork 38
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-19: remove -
from cpu names
#252
Comments
compiler-errors
added a commit
to compiler-errors/rust
that referenced
this issue
Mar 6, 2025
…=jieyouxu Remove - from xtensa targets cpu names This PR removes the `-` from the Xtensa CPU names as it was updated in LLVM, see esp-rs#252
rust-timer
added a commit
to rust-lang-ci/rust
that referenced
this issue
Mar 7, 2025
Rollup merge of rust-lang#138064 - SergioGasquez:feat/xtensa-names, r=jieyouxu Remove - from xtensa targets cpu names This PR removes the `-` from the Xtensa CPU names as it was updated in LLVM, see esp-rs#252
fc08bef implmented this on our fork and rust-lang#138064 just got merged upstream! So one less patch to take care about |
lnicola
pushed a commit
to lnicola/rust-analyzer
that referenced
this issue
Mar 10, 2025
Remove - from xtensa targets cpu names This PR removes the `-` from the Xtensa CPU names as it was updated in LLVM, see esp-rs/rust#252
github-actions bot
pushed a commit
to rust-lang/miri
that referenced
this issue
Mar 11, 2025
Remove - from xtensa targets cpu names This PR removes the `-` from the Xtensa CPU names as it was updated in LLVM, see esp-rs/rust#252
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In LLVM19 when we upgrade this fork to it, we must update the targets to use
esp32s3
notesp32-s3
. We must also reflect this upstream.The text was updated successfully, but these errors were encountered: