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

[ROCm][Windows] Enable build with ROCm on Windows #3883

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tvukovic-amd
Copy link

@tvukovic-amd tvukovic-amd commented Feb 19, 2025

Added changes for Windows in LoadHIP.cmake:

  • set default ROCM_PATH and HIP_PATH for Windows case
  • update CMAKE_MODULE_PATH with appropriate HIP path - because the find_package call to this module uses the Module mode search
  • added code for finding HIP version
  • skipped find hsa-runtime64, rccl packages on Windows case
  • skipped find rccl, roctx and roctracer libraries on Windows case

CMAKE_C_COMPILER and CMAKE_CXX_COMPILER redefinition on Windows case (if we are building with ROCm on Windows we need to use compiler specified with CXX and CC flags instead of cl)

@tvukovic-amd tvukovic-amd requested a review from a team as a code owner February 19, 2025 10:07
Copy link

pytorch-bot bot commented Feb 19, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/audio/3883

Note: Links to docs will display an error until the docs builds have been completed.

This comment was automatically generated by Dr. CI and updates every 15 minutes.

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

Successfully merging this pull request may close these issues.

2 participants