Skip to content

[ROCm][Windows] Fixing undefined symbol linker error after exposing MIOpen symbols #158918

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

Closed

Conversation

tvukovic-amd
Copy link
Contributor

@tvukovic-amd tvukovic-amd commented Jul 23, 2025

Changing TORCH_CUDA_CPP_API macros to TORCH_HIP_CPP in MIOpen header files according to closed PR in pytorch upstream.

cc @jeffdaily @sunway513 @jithunnair-amd @pruthvistony @ROCmSupport @dllehr-amd @jataylo @hongxiayang @naromero77amd

Copy link

pytorch-bot bot commented Jul 23, 2025

🔗 Helpful Links

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

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

❌ 29 Cancelled Jobs

As of commit 3ca3354 with merge base 3a7ff82 (image):

CANCELLED JOBS - The following jobs were cancelled. Please retry:

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

@pytorch-bot pytorch-bot bot added the module: rocm AMD GPU support for Pytorch label Jul 23, 2025
…IOpen symbols (pytorch#156479)

Fixing undefined symbol linker error after [exposing MIOpen symbols](pytorch#154545).
This fix:

- Hipifies `aten/src/ATen/miopen` and `aten/src/ATen/native/miopen` files
- Adds `aten/src/ATen/miopen` and `aten/src/ATen/native/miopen` hipified source files to `all_hip_cpp` list

Pull Request resolved: pytorch#156479
Approved by: https://github.com/jeffdaily

Co-authored-by: Jeff Daily <[email protected]>
Copy link
Collaborator

@jeffdaily jeffdaily left a comment

Choose a reason for hiding this comment

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

These are good changes but release/2.8 is no longer accepting fixes. Consider this for a potential 2.8.1 patch release or please re-submit to the ROCm fork of pytorch release/2.8 where we can make off-cycle changes like these.

@jeffdaily jeffdaily closed this Jul 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: rocm AMD GPU support for Pytorch open source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants