Skip to content

Stop assuming Windows in the build configuration - #151

Closed
mischa85 wants to merge 8 commits into
OpenTS-Developers:mainfrom
mischa85:portable-test-harness
Closed

Stop assuming Windows in the build configuration#151
mischa85 wants to merge 8 commits into
OpenTS-Developers:mainfrom
mischa85:portable-test-harness

Conversation

@mischa85

@mischa85 mischa85 commented Sep 9, 2026

Copy link
Copy Markdown

opents_add_test() and code/CMakeLists.txt hand MSVC's options to every compiler, so nothing in the tree builds natively. This gives them a native equivalent behind OPENTS_EXPERIMENTAL_NATIVE, mirroring OPENTS_EXPERIMENTAL_X64.

Sixteen of the thirty-seven harnesses now build and pass off Windows, with a Linux job to keep them that way. The engine still does not link, but 420 of its translation units compile where none did. The remaining 323 stop on windows.h, comdef.h and io.h. A harness that cannot build yet declares UNPORTED.

The supported build is unchanged: the same MSVC options, only guarded. /fp:precise maps to -ffp-contract=off and /arch:SSE2 to -msse2 -mfpmath=sse on 32-bit x86. /RTC1 and the static runtime have no equivalent.

@mischa85
mischa85 marked this pull request as draft September 9, 2026 11:01
@mischa85 mischa85 closed this Sep 9, 2026
@mischa85
mischa85 deleted the portable-test-harness branch September 9, 2026 14:14
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