Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][CUDA] Added tests for atomic loads and stores for various orders and scopes #648

Merged
merged 9 commits into from
Mar 14, 2022

Conversation

t4c1
Copy link

@t4c1 t4c1 commented Dec 21, 2021

Added tests for atomic loads and stores for various orders and scopes. This is testing changes in: intel/llvm#5191.

This is similar kind of change as in #534.

Blocked by: intel/llvm#4853

@t4c1 t4c1 requested a review from AGindinson as a code owner December 21, 2021 08:57
@t4c1 t4c1 requested a review from a team as a code owner January 6, 2022 08:36
@t4c1 t4c1 requested a review from v-klochkov January 6, 2022 08:36
bader pushed a commit to intel/llvm that referenced this pull request Jan 17, 2022
…orders and scopes (#5191)

Adds atomic loads and stores with various memory orders and scopes. These are implemented by adding intrinsics and builtins for   PTX loads and stores.

Tests for this are here: intel/llvm-test-suite#648
@t4c1 t4c1 requested a review from steffenlarsen February 22, 2022 12:13
@t4c1
Copy link
Author

t4c1 commented Mar 2, 2022

@steffenlarsen Could I get a review for this PR?

steffenlarsen
steffenlarsen previously approved these changes Mar 2, 2022
Copy link

@steffenlarsen steffenlarsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@vladimirlaz
Copy link

@t4c1, could you please resolve conflicts?

@vladimirlaz vladimirlaz merged commit d944065 into intel:intel Mar 14, 2022
@t4c1 t4c1 deleted the ptx_atomic_ldst branch March 15, 2022 08:52
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Apr 12, 2022
myler pushed a commit to myler/llvm-test-suite that referenced this pull request Jun 17, 2022
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants