Skip to content

Adding missing __shfl overloads.#557

Closed
Jorghi12 wants to merge 2 commits into
ROCm:roc-1.8.xfrom
Jorghi12:patch-7
Closed

Adding missing __shfl overloads.#557
Jorghi12 wants to merge 2 commits into
ROCm:roc-1.8.xfrom
Jorghi12:patch-7

Conversation

@Jorghi12
Copy link
Copy Markdown
Contributor

@Jorghi12 Jorghi12 commented Jul 6, 2018

Added missing double overloads for __shfl operations.

Jorghi12 and others added 2 commits July 6, 2018 19:03
Added missing double overloads for __shfl operations.
Added definitions for __shfl double operations.
@Jorghi12 Jorghi12 changed the title Adding missing overloads. Adding missing __shfl overloads. Jul 7, 2018
@dagamayank
Copy link
Copy Markdown

/cc @mangupta

@AlexVlx
Copy link
Copy Markdown
Contributor

AlexVlx commented Jul 10, 2018

@Jorghi12 please don't do it like so, we are in the process of moving most of HIP to being header only. please coordinate with @aaronenyeshi who is driving this to get the overloads you are missing into the right place.

@mangupta
Copy link
Copy Markdown
Contributor

FYI the PR that Alex mentioned is PR #515.

@aaronenyeshi
Copy link
Copy Markdown
Contributor

Hi @Jorghi12 I've added the missing double __shfl functions in that PR 9ac31e0

@mangupta
Copy link
Copy Markdown
Contributor

Closing this since PR#515 now contains the changes that were required by this PR as well.

@mangupta mangupta closed this Jul 11, 2018
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.

5 participants