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

Merge atomic load/store into regular load/store #559

Closed
wants to merge 2 commits into from
Closed

Merge atomic load/store into regular load/store #559

wants to merge 2 commits into from

Conversation

ssikdar1
Copy link
Contributor

Description

Closes #450

Merges atomic_load into load and atomic_store into store

Testing

Ran llvmlite.tests

$ python -m llvmlite.tests
..s................................................................................................................................................................................................................................s..........................
----------------------------------------------------------------------
Ran 254 tests in 1.831s

OK (skipped=2)

@stuartarchibald
Copy link
Contributor

Thanks for the patch. Note to reviewers, this is a public API change.

@stuartarchibald stuartarchibald added this to the Version 0.33.0 milestone Mar 23, 2020
@ARF1 ARF1 mentioned this pull request Mar 28, 2020
Open
@sklam sklam modified the milestones: Version 0.33.0, Version 0.34.0 Jun 3, 2020
@stuartarchibald stuartarchibald removed this from the Version 0.35.0 milestone Nov 6, 2020
@esc
Copy link
Member

esc commented Mar 22, 2021

Closing in favour of #564

@esc esc closed this Mar 22, 2021
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
3 - Ready for Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Merge atomic load/store into regular load/store
4 participants
@ssikdar1 @stuartarchibald @esc @sklam