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

Add atomic functions for unsigned long long using gcc built-in #3588

Merged
merged 1 commit into from
Nov 13, 2020

Conversation

Rombur
Copy link
Member

@Rombur Rombur commented Nov 12, 2020

This fixes issue #3581 for me. I don't know why these overloads were missing since they are supported (see here).

I am not sure of the exact problem in #3581 because I couldn't create a simple reproducer.

@Rombur Rombur added the Blocks Promotion Overview issue for release-blocking bugs label Nov 12, 2020
@dalg24
Copy link
Member

dalg24 commented Nov 13, 2020

I would like to discuss adding unit tests before we merge
Christian convinced me that it was wasted effort before because #3247 was going to cover this.

@dalg24
Copy link
Member

dalg24 commented Nov 13, 2020

I verified that this resolves the issue I saw in ArborX callback example.

@dalg24 dalg24 merged commit 2cfb930 into kokkos:develop Nov 13, 2020
masterleinad added a commit to masterleinad/kokkos that referenced this pull request Nov 17, 2020
This reverts commit 2cfb930, reversing
changes made to e34d3bc.
@Rombur Rombur deleted the fix_rocm39 branch November 24, 2020 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocks Promotion Overview issue for release-blocking bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants