Skip to content
This repository has been archived by the owner on Jul 26, 2024. It is now read-only.

HIP version of kronmult #13

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

ckendrick
Copy link

This replaces CUDA with HIP and adjusts CMakeLists.txt for compiling on both AMD and NVIDIA platforms.

Copy link
Contributor

@quantumsteve quantumsteve left a comment

Choose a reason for hiding this comment

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

Please double-check that we need find_package(CUDA).

CMakeLists.txt Outdated
message(STATUS "HIP Libraries: ${HIP_LIBRARIES}")

if(KRONMULT_PLATFORM_NVCC)
find_package(CUDA 9.0 REQUIRED)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is a step back from enable_language (CUDA). Can we stay with first-class language support?

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.

2 participants