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.

Sign up for GitHub

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

Jump to bottom

Properly support <atomic> on AVR and Cortex-M #1164

Merged
merged 6 commits into from
May 20, 2024
Merged

Properly support <atomic> on AVR and Cortex-M #1164

merged 6 commits into from
May 20, 2024

Conversation

salkinium
Copy link
Member

@salkinium salkinium commented May 19, 2024 •
edited
Loading

  • Implement all atomics library calls for AVR and ARM Cortex-M
  • Add atomic headers back avr-libstdcpp#34
  • Add unit test for <atomic>
  • Hardware testing
    • STM32H7
    • STM32G0
    • ATmega2560
  • Inline most of the atomic library calls for significant overhead reduction
  • Correct memory order support for single- and multi-core devices
  • Fix the static initialization guard (again)
  • Documentation

@salkinium salkinium added this to the 2024q2 milestone May 19, 2024
@salkinium salkinium force-pushed the fix/atomics branch 3 times, most recently from 5a2ad07 to 7c5a2a8 Compare May 19, 2024 22:12
@salkinium salkinium marked this pull request as ready for review May 20, 2024 10:16
@salkinium salkinium added the ci:hal Triggers the exhaustive HAL compile CI jobs label May 20, 2024
@salkinium salkinium merged commit 8a924f9 into modm-io:develop May 20, 2024
38 of 39 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
advanced 🤯 ci:hal Triggers the exhaustive HAL compile CI jobs enhancement 🌈 fix 💎
Milestone
2024q2
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@salkinium