Skip to content

Commit

Permalink
Rollup merge of #133599 - esp-rs:target/esp32s2-forced-atomics, r=Ama…
Browse files Browse the repository at this point in the history
…nieu

Add `+forced-atomics` feature to esp32s2 no_std  target

Similar to #114499 but for the Xtensa backend. The ESP32-S2 doesn't have native atomic support, but can have atomic load/stores as part of the ISA with this LLVM codegen feature.

Note: The current rev of LLVM that rustc is using doesn't contain the `+forced-atomics` feature for Xtensa, but I'm pushing this now to remove the patch from our fork in `esp-rs/rust`.

r? ``@Amanieu`` because you reviewed the related RISC-V PR
  • Loading branch information
jieyouxu authored Nov 30, 2024
2 parents ab4588a + 727f6a6 commit 46f826c
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ pub(crate) fn target() -> Target {
cpu: "esp32-s2".into(),
linker: Some("xtensa-esp32s2-elf-gcc".into()),
max_atomic_width: Some(32),
features: "+forced-atomics".into(),
..xtensa::opts()
},
}
Expand Down

0 comments on commit 46f826c

Please sign in to comment.