Skip to content

<atomic>: Workaround __iso_volatile_{load,store}64 being non-atomic with clang-cl on 32-bit x86 - #5708

Merged
Stephan T. Lavavej (StephanTLavavej) merged 2 commits into
microsoft:mainfrom
cpplearner:patch-1
Sep 10, 2025
Merged

Conversation

@cpplearner

Copy link
Copy Markdown
Contributor

Closes #5705

Comment thread stl/inc/atomic Outdated
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) added bug Something isn't working compiler Compiler work involved labels Sep 5, 2025
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Initial Review to Ready To Merge in STL Code Reviews Sep 5, 2025
Comment thread stl/inc/atomic
@StephanTLavavej Stephan T. Lavavej (StephanTLavavej) moved this from Ready To Merge to Merging in STL Code Reviews Sep 10, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed

@StephanTLavavej
Stephan T. Lavavej (StephanTLavavej) merged commit 8fa0bc8 into microsoft:main Sep 10, 2025
39 checks passed
@github-project-automation github-project-automation Bot moved this from Merging to Done in STL Code Reviews Sep 10, 2025
@StephanTLavavej

Copy link
Copy Markdown
Member

Thanks for adding this compiler bug workaround - they were starting to become an endangered species! 🦓 🪲 📉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working compiler Compiler work involved

3 participants