Skip to content

Conversation

@tvukovic-amd
Copy link
Contributor

@tvukovic-amd tvukovic-amd commented Oct 15, 2025

AOTriton uses prebuilt runtime binaries if the user's ROCm version matches the ones used to generate the prebuilt runtime. However, since there's no prebuilt runtime available for Windows, this check needs to be bypassed for Windows. This PR enables it by changing condition to always build AOTriton runtime from source on Windows.

cc @peterjc123 @mszhanyi @skyline75489 @nbcsm @iremyux @Blackhex @jeffdaily @sunway513 @jithunnair-amd @pruthvistony @ROCmSupport @dllehr-amd @jataylo @hongxiayang @naromero77amd

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 15, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (2 Unrelated Failures)

As of commit 88ffbdf with merge base 0c14f55 (image):

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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 Oct 15, 2025
@jammm
Copy link
Contributor

jammm commented Oct 15, 2025

@pytorchbot label "release notes: rocm"

@jammm
Copy link
Contributor

jammm commented Oct 15, 2025

@pytorchbot label "module: rocm"

@pytorch-bot pytorch-bot bot added the release notes: rocm mandatorylabel label Oct 15, 2025
@jammm
Copy link
Contributor

jammm commented Oct 15, 2025

@pytorchbot label "module: windows"

@pytorch-bot pytorch-bot bot added the module: windows Windows support for PyTorch label Oct 15, 2025
@jammm
Copy link
Contributor

jammm commented Oct 15, 2025

cc @xinyazhang

Copy link
Collaborator

@xinyazhang xinyazhang left a comment

Choose a reason for hiding this comment

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

Although the ideal solution is matching the manylinux_2_28_x86_64 substring in the package name or using a catalog file signed with sigstore, but it is good enough for now.

@tvukovic-amd tvukovic-amd force-pushed the build_aotriton_runtime_on_windows branch from ad75f32 to 88ffbdf Compare October 16, 2025 07:43
@jeffdaily
Copy link
Collaborator

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 16, 2025
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

tvukovic-amd added a commit to ROCm/pytorch that referenced this pull request Oct 17, 2025
…65538)

AOTriton uses prebuilt runtime binaries if the user's ROCm version matches the ones used to generate the prebuilt runtime. However, since there's no prebuilt runtime available for Windows, this check needs to be bypassed for Windows. This PR enables it by changing condition to always build AOTriton runtime from source on Windows.

Pull Request resolved: pytorch#165538
Approved by: https://github.com/xinyazhang, https://github.com/jeffdaily
Chao1Han pushed a commit to Chao1Han/pytorch that referenced this pull request Oct 21, 2025
…65538)

AOTriton uses prebuilt runtime binaries if the user's ROCm version matches the ones used to generate the prebuilt runtime. However, since there's no prebuilt runtime available for Windows, this check needs to be bypassed for Windows. This PR enables it by changing condition to always build AOTriton runtime from source on Windows.

Pull Request resolved: pytorch#165538
Approved by: https://github.com/xinyazhang, https://github.com/jeffdaily
zhudada0120 pushed a commit to zhudada0120/pytorch that referenced this pull request Oct 22, 2025
…65538)

AOTriton uses prebuilt runtime binaries if the user's ROCm version matches the ones used to generate the prebuilt runtime. However, since there's no prebuilt runtime available for Windows, this check needs to be bypassed for Windows. This PR enables it by changing condition to always build AOTriton runtime from source on Windows.

Pull Request resolved: pytorch#165538
Approved by: https://github.com/xinyazhang, https://github.com/jeffdaily
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged module: rocm AMD GPU support for Pytorch module: windows Windows support for PyTorch open source release notes: rocm mandatorylabel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants