Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

jeffdaily
Copy link
Contributor

No description provided.

@jeffdaily jeffdaily changed the title add ROCm 3.8 manywheel build add ROCm 3.9 manywheel build Oct 29, 2020
@jeffdaily
Copy link
Contributor Author

@walterddr could you please review? Thanks.

if [[ $ROCM_VERSION == "rocm3.7" ]]; then
TENSILE_LIBRARY_NAME=TensileLibrary.yaml
elif [[ $ROCM_VERSION == "rocm3.8" ]]; then
else
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
else
elif [[ $ROCM_VERSION == "rocm3.8" || $ROCM_VERSION == "rocm3.9" ]]; then

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did consider that, but IIUC the tensile library name extension will hereafter always be *.dat, so the rocm3.7 case was the only exceptional case.

Copy link
Contributor

Choose a reason for hiding this comment

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

got it. thanks for the explanation

@walterddr walterddr merged commit a15fac8 into pytorch:master Oct 30, 2020
@walterddr
Copy link
Contributor

@jeffdaily did you have rocm3.9 docker images created?

facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Nov 2, 2020
Summary:
Corresponding pytorch builder repo update: pytorch/builder#561.

Pull Request resolved: #47121

Reviewed By: samestep

Differential Revision: D24660850

Pulled By: walterddr

fbshipit-source-id: 68b22e0a2d341396eb1cdcfaa0a413ce7ad93ca3
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