Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[build.ps1] Enable LLDB tests in Windows x86_64 builds. #75946

Closed
wants to merge 2 commits into from

Conversation

kendalharland
Copy link

Related to swiftlang/llvm-project#9141

This PR enables the LLDB test suite on Windows builds of the swift toolchain. As a reference, I've built and run all of the LLDB llvm/llvm-project tests locally (There is not Windows x86_64 CI for upstream LLVM, only aarch64) and all of the tests pass without modification. However, in this fork, many of the tests fail. This hints that the failures are due to changes contained in this fork that must be addressed. To avoid handling these all at once, a file at utils/windows-llvm-lit-test-overrides.txt is introduced which marks tests as XFAIL or skipped (if flaky) with tracking issues to provide us with a list of tests to fix.

kendal added 2 commits August 17, 2024 13:17
This commit runs the LLDB test suite when building llvm-project.
Many of the tests are already failing. These are either skipped
or marked as xfail (xfail preferred where possible) and tracked
in utils/windows_lit_test_overrides.txt. utils/build.ps1 will
use this list to override the test's status when llvm-lit.py
is invoked.
@kendalharland kendalharland marked this pull request as draft August 17, 2024 20:33
@compnerd
Copy link
Member

@swift-ci please test Windows platform

@compnerd
Copy link
Member

@swift-ci please test

@compnerd
Copy link
Member

@swift-ci please test Windows platform

@compnerd compnerd marked this pull request as ready for review September 13, 2024 04:47
@compnerd
Copy link
Member

@swift-ci please test Windows platform

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.

2 participants