Skip to content

Add support for building CTS natively on Windows#2722

Open
jujiang-del wants to merge 9 commits into
KhronosGroup:mainfrom
jujiang-del:jujiang/gcc
Open

Add support for building CTS natively on Windows#2722
jujiang-del wants to merge 9 commits into
KhronosGroup:mainfrom
jujiang-del:jujiang/gcc

Conversation

@jujiang-del

Copy link
Copy Markdown
Contributor

Adding instructions and support to build CTS on Windows using MSYS2 gcc.

@rjodinchr rjodinchr self-requested a review June 12, 2026 05:13
Comment thread test_conformance/api/test_api_min_max.cpp
Comment thread README.md
Comment on lines +107 to +111
### Building CTS on Windows

For Windows environments, it is strongly recommended to build CTS using [MSYS2](https://www.msys2.org/),
the MinGW-w64 (GCC) toolchain, and Ninja.
All commands in the following sections should be run from an MSYS2 MinGW64 shell.

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.

Can you give use more information why that is?

@jujiang-del jujiang-del Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

For example, using the MSVC compiler on Windows, in which long double is treated as identical to double, so it eliminates extended‑precision support and can introduce precision loss in tests that rely on higher‑precision floating‑point behavior, such as brute_math FP64 for asinh, atanh, , expm1, tgamma , etc.

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.

I'm having trouble thinking that this justifies the OpenCL group recommending MSY2 over anything else.
But I'm not a Windows user, so I will let that to people with more expertise.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If we want to advertise support for MYS2, regardless of whether we recommend it over MSVC on Windows, I think we should introduce CI testing for that configuration. MSVC builds on windows are already tested in CI, but without CI testing we could easily break a MYS2 build again after this PR is merged.

@rjodinchr rjodinchr requested review from rjodinchr and removed request for rjodinchr June 12, 2026 05:13
@jujiang-del jujiang-del requested a review from rjodinchr June 30, 2026 20:29
@rjodinchr rjodinchr removed their request for review July 1, 2026 06:04

@rjodinchr rjodinchr 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.

Fine as far as I can see. We need some people involved with windows to review

@rjodinchr rjodinchr self-requested a review July 1, 2026 06:06
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.

3 participants