This is an archive of the discontinued LLVM Phabricator instance.

[clang] Add a Windows build in the Clang pre-commit CI
ClosedPublic

Authored by ldionne on Aug 28 2023, 8:00 AM.

Details

Summary

This patch adds a CI job for Clang on Windows that is separate from
the monolithic job that gets added automatically via the Phabricator
integration with Buildkite. This way, we will retain the Windows testing
for Clang when we move to GitHub Pull Requests.

Diff Detail

Event Timeline

ldionne created this revision.Aug 28 2023, 8:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 28 2023, 8:00 AM
ldionne requested review of this revision.Aug 28 2023, 8:00 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 28 2023, 8:00 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript

@aaron.ballman for awareness

Thank you for this!

ldionne updated this revision to Diff 553968.Aug 28 2023, 9:36 AM

Run script through bash on Windows.

ldionne updated this revision to Diff 554015.Aug 28 2023, 12:40 PM

Set VC variables

ldionne updated this revision to Diff 554115.Aug 28 2023, 5:27 PM

Poke CI -- it seemed to time out this afternoon due to Phab.

ldionne updated this revision to Diff 554277.Aug 29 2023, 5:47 AM

Don't upload the Windows build artifacts cause we don't use them and it fails.

ldionne updated this revision to Diff 554295.Aug 29 2023, 6:27 AM

Poke CI -- Phabricator keeps timing out.

This looks good, the Windows CI is failing with the original premerge checks in exactly the same way as this new job, so I guess it's a current issue on main.

This revision was not accepted when it landed; it landed in state Needs Review.Aug 30 2023, 6:17 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.