Skip to content

Enable hipTensor examples on Windows#458

Open
mapatel-amd wants to merge 1 commit into
amd-stagingfrom
mapatel/hiptensor-arch-check
Open

Enable hipTensor examples on Windows#458
mapatel-amd wants to merge 1 commit into
amd-stagingfrom
mapatel/hiptensor-arch-check

Conversation

@mapatel-amd
Copy link
Copy Markdown
Contributor

@mapatel-amd mapatel-amd commented May 28, 2026

Motivation

hipTensor now supports Windows. The previous PR #454 incorrectly blocked building hipTensor examples on Windows with a Linux-only guard. This PR removes that restriction.

Technical Details

  • Remove if(CMAKE_SYSTEM_NAME MATCHES "Windows") ... return() endif() guards from all 21 hipTensor CMakeLists files
  • Add install(IMPORTED_RUNTIME_ARTIFACTS hiptensor) for Windows in each leaf CMakeLists
  • Hoist ROCmPath.cmake include to before find_package(hiptensor) in all leaf files, so CMAKE_PREFIX_PATH is populated before package discovery on Windows

Test Plan

  • Linux: existing CI coverage
  • Windows: hipTensor examples should now configure and build

Submission Checklist

hipTensor now supports Windows. Remove the Linux-only early-return guards
and add install(IMPORTED_RUNTIME_ARTIFACTS hiptensor) for Windows. Also
hoist ROCmPath.cmake to before find_package(hiptensor) so CMAKE_PREFIX_PATH
is populated before package discovery on Windows.

Follows up on #454.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
@mapatel-amd mapatel-amd requested a review from a team as a code owner May 28, 2026 20:54
mapatel-amd added a commit that referenced this pull request May 28, 2026
hipSPARSELt supports Windows. Remove the Linux-only guards and add
install(IMPORTED_RUNTIME_ARTIFACTS roc::hipsparselt) for Windows,
following the same pattern applied to hipTensor in #458.

Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com>
@mapatel-amd mapatel-amd self-assigned this May 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant