Skip to content

[CIR][CIRGen][Builtin] Support unsigned type for _sync_(bool/val)_compare_and_swap #955

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

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

ghehg
Copy link
Collaborator

@ghehg ghehg commented Oct 9, 2024

Actually just follow the way in makeBinaryAtomicValue in the same file which did the right thing by creating SInt or UInt based on first argument's signess.

@ghehg ghehg marked this pull request as ready for review October 9, 2024 17:53
@bcardosolopes bcardosolopes merged commit 07897b3 into llvm:main Oct 9, 2024
9 checks passed
smeenai pushed a commit to smeenai/clangir that referenced this pull request Oct 9, 2024
…pare_and_swap (llvm#955)

as title. 
Actually just follow the way in `makeBinaryAtomicValue` in the same file
which did the right thing by creating SInt or UInt based on first
argument's signess.
keryell pushed a commit to keryell/clangir that referenced this pull request Oct 19, 2024
…pare_and_swap (llvm#955)

as title. 
Actually just follow the way in `makeBinaryAtomicValue` in the same file
which did the right thing by creating SInt or UInt based on first
argument's signess.
lanza pushed a commit that referenced this pull request Nov 5, 2024
…pare_and_swap (#955)

as title. 
Actually just follow the way in `makeBinaryAtomicValue` in the same file
which did the right thing by creating SInt or UInt based on first
argument's signess.
lanza pushed a commit that referenced this pull request Mar 18, 2025
…pare_and_swap (#955)

as title. 
Actually just follow the way in `makeBinaryAtomicValue` in the same file
which did the right thing by creating SInt or UInt based on first
argument's signess.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants