Skip to content

Add Windows CI for verifying and building Visual Studio solutions#244

Merged
zichguan-amd merged 1 commit into
amd-stagingfrom
zichguan/windows-ci
Jun 4, 2025
Merged

Add Windows CI for verifying and building Visual Studio solutions#244
zichguan-amd merged 1 commit into
amd-stagingfrom
zichguan/windows-ci

Conversation

@zichguan-amd

@zichguan-amd zichguan-amd commented May 29, 2025

Copy link
Copy Markdown
Collaborator

Adds a GitHub action using Github-hosted Windows runner

  • Update and use check_vs_files.py script to validate GUIDs in top-level solution files
  • Adds build.proj project file to find and build individual example's .sln file using msbuild (task batch parallel build)
  • Excludes vulkan_interop, opengl_interop, and static_host_library examples due to additional dependency requirements
  • Build triggers independently for each category (Applications, HIP-Basic, Libraries)
  • Builds both Debug and Release on x64 in parallel using matrix strategy
  • HIP SDK installation will take ~15 to 20 mins
  • For Libraries, parallelize build further for vs2017, 2019, and 2022 to reduce runtime to ~20min

@zichguan-amd zichguan-amd requested a review from a team as a code owner May 29, 2025 15:31
@zichguan-amd zichguan-amd force-pushed the zichguan/windows-ci branch from 0757afe to dc3806d Compare May 29, 2025 15:34

@tcgu-amd tcgu-amd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@zichguan-amd zichguan-amd force-pushed the zichguan/windows-ci branch 6 times, most recently from 64ff9af to ece9636 Compare June 4, 2025 20:31
Signed-off-by: zichguan-amd <zichuan.guan@amd.com>
@zichguan-amd zichguan-amd force-pushed the zichguan/windows-ci branch from ece9636 to 7088d84 Compare June 4, 2025 20:33
@zichguan-amd zichguan-amd merged commit 3c58654 into amd-staging Jun 4, 2025
21 of 23 checks passed
@zichguan-amd zichguan-amd deleted the zichguan/windows-ci branch June 5, 2025 13:59
@zichguan-amd zichguan-amd added the continuous integration changes to workflows label Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

continuous integration changes to workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants