Skip to content

[SYCL][CUDA][libclc] Add remaining atomics #5192

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 5 commits into from
Jan 18, 2022

Conversation

t4c1
Copy link
Contributor

@t4c1 t4c1 commented Dec 21, 2021

Adds implementations for atomic operations that do not have direct PTX equivalents. Sub is implemented by using add. Floating point min and max use compare exchange loops.

I pushed the tests for this into: intel/llvm-test-suite#534

This blocks: #4853

@t4c1 t4c1 requested a review from bader as a code owner December 21, 2021 09:31
@bader
Copy link
Contributor

bader commented Dec 21, 2021

/verify with intel/llvm-test-suite#534

bader
bader previously approved these changes Dec 21, 2021
@bader
Copy link
Contributor

bader commented Dec 23, 2021

/verify with intel/llvm-test-suite#534

@t4c1
Copy link
Contributor Author

t4c1 commented Jan 17, 2022

@bader I think this one can be merged as well. It should pass the test suite on intel branch, while intel/llvm-test-suite#534 will require #4853 merged before.

@bader
Copy link
Contributor

bader commented Jan 17, 2022

/verify with intel/llvm-test-suite#697

@t4c1
Copy link
Contributor Author

t4c1 commented Jan 18, 2022

I think the failures are unrelated to the changes in this PR.

@bader bader merged commit 6b2635e into intel:sycl Jan 18, 2022
bader pushed a commit to intel/llvm-test-suite that referenced this pull request Feb 24, 2022
)

Added tests for atomics with various memory orders and scopes. Reductions tests also have updated sm requirements, as they call work group atomics, which are now implemented and have higher sm requirements than device scoped ones.

This adds tests for changes introduced in intel/llvm#4820 and intel/llvm#5192.
@t4c1 t4c1 deleted the remaining_ptx_atomics branch March 15, 2022 08:51
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
…ntel/llvm-test-suite#534)

Added tests for atomics with various memory orders and scopes. Reductions tests also have updated sm requirements, as they call work group atomics, which are now implemented and have higher sm requirements than device scoped ones.

This adds tests for changes introduced in intel#4820 and intel#5192.
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