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.

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

Convert AtomicObjects to use compareAndSwap instead of compareExchange #14008

Merged
merged 1 commit into from
Sep 5, 2019

Conversation

ronawho
Copy link
Contributor

@ronawho ronawho commented Sep 5, 2019

Update AtomicObjects to match the new Chapel API (changed in #13934)

This change was discussed a little in #13836 (comment)

Update AtomicObjects to match the new Chapel API.
@ronawho
Copy link
Contributor Author

ronawho commented Sep 5, 2019

@LouisJenkinsCS mind taking a look?

@LouisJenkinsCS
Copy link
Member

Looks good so far. I can add compareExchange that updates the expected myself later.

@ronawho ronawho merged commit 818cc53 into chapel-lang:master Sep 5, 2019
@ronawho ronawho deleted the AtomicObjects-CAS branch September 5, 2019 19:00
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