Skip to content

First version for support on AMD GPU via HIP backend - #3496

Open
jonycastagna wants to merge 5 commits into
nextfrom
next-gpu-amd
Open

jonycastagna wants to merge 5 commits into
nextfrom
next-gpu-amd

Conversation

@jonycastagna

Copy link
Copy Markdown

Added HIP directive for offloading on AMD GPU via RAJA and binary expression backend

@ZedThree ZedThree left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, I've got some questions but they might not be blockers

Comment thread tests/MMS/diffusion/diffusion.cxx
Comment thread tests/MMS/advection/CMakeLists.txt
Comment thread src/mesh/parallel/shiftedmetric.cxx
Comment thread cmake/SetupCompilers.cmake
Comment thread cmake/SetupBOUTThirdParty.cmake
@jonycastagna

Copy link
Copy Markdown
Author

Hi @ZedThree, I just added some comments above and merged latest from "next" but seems some checks were not succesful. Do you know what I am doing wrong? Thanks!

@ZedThree

Copy link
Copy Markdown
Member

The failing test is unrelated

@jonycastagna

Copy link
Copy Markdown
Author

Ok, thanks! I see now is all good

@ZedThree

Copy link
Copy Markdown
Member

@jonycastagna Is this ready to merge? You mentioned some other work -- is that a second PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants