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

atomics: optimize modify operation (partially) #41859

Merged
merged 2 commits into from
Aug 26, 2021
Merged

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented Aug 10, 2021

Optimize the load/store portion of the operations, but not yet the
invoke part. (partially addressing #41843)

Optimize the load/store portion of the operations, but not yet the
invoke part.
@vtjnash vtjnash added performance Must go faster multithreading Base.Threads and related functionality backport 1.7 labels Aug 10, 2021
src/cgutils.cpp Outdated Show resolved Hide resolved
@vtjnash vtjnash added the merge me PR is reviewed. Merge when all tests are passing label Aug 25, 2021
@vtjnash vtjnash merged commit 690eae2 into master Aug 26, 2021
@vtjnash vtjnash deleted the jn/optimize-atomics-2 branch August 26, 2021 16:08
KristofferC pushed a commit that referenced this pull request Aug 27, 2021
Optimize the load/store portion of the operations, but not yet the
invoke part.

(cherry picked from commit 690eae2)
@DilumAluthge DilumAluthge removed the merge me PR is reviewed. Merge when all tests are passing label Sep 6, 2021
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Feb 22, 2022
Optimize the load/store portion of the operations, but not yet the
invoke part.
LilithHafner pushed a commit to LilithHafner/julia that referenced this pull request Mar 8, 2022
Optimize the load/store portion of the operations, but not yet the
invoke part.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multithreading Base.Threads and related functionality performance Must go faster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants