Perform ThinLTO on x86_64-pc-windows-msvc dist builder#128947
Perform ThinLTO on x86_64-pc-windows-msvc dist builder#128947Zoxc wants to merge 1 commit intorust-lang:mainfrom
Conversation
|
FYI @rust-lang/infra |
|
☔ The latest upstream changes (presumably #130506) made this pull request unmergeable. Please resolve the merge conflicts. |
|
@rustbot ready This is no longer blocked, at least not on my previous PR. Are the binaries built by dist jobs properly tested now? |
|
They are tested, not sure if properly :) Last time we tried it, one test was still miscompiled, we could try again.
|
EDIT: x86_64-pc-windows-msvc goes through opt-dist but not the other i686-pc-windows-msvc apparently. Recently we broke i686-pc-windows-gnu rustc binaries in a way that caused rustc binary to completely not work (rustc -vV failed) but no tests caught it. |
We do test dist x64 msvc artifacts in opt-dist. |
|
Huh. I must be thinking of x86_64 windows-gnu or maybe the 32 bit windows-msvc target? EDIT: in any case you're right: https://github.com/rust-lang/rust/blob/master/src%2Fci%2Fgithub-actions%2Fjobs.yml#L584 |
|
@bors try |
Perform ThinLTO on x86_64-pc-windows-msvc dist builder This reenables ThinLTO on x86_64-pc-windows-msvc. Depends on rust-lang#122790 try-job: dist-x86_64-msvc
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
|
@bors try |
Perform ThinLTO on x86_64-pc-windows-msvc dist builder This reenables ThinLTO on x86_64-pc-windows-msvc. Depends on rust-lang#122790 try-job: dist-x86_64-msvc
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
|
☔ The latest upstream changes (presumably #138630) made this pull request unmergeable. Please resolve the merge conflicts. |
|
The error looks like memory unsafety, possibly due to a miscompilation. @bors try |
Perform ThinLTO on x86_64-pc-windows-msvc dist builder This reenables ThinLTO on x86_64-pc-windows-msvc. try-job: dist-x86_64-msvc
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
|
☔ The latest upstream changes (presumably #145295) made this pull request unmergeable. Please resolve the merge conflicts. |
|
This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
@bors try |
Perform ThinLTO on x86_64-pc-windows-msvc dist builder try-job: dist-x86_64-msvc
This comment has been minimized.
This comment has been minimized.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for 74ab0d8 failed: CI. Failed jobs:
|
|
@bors try |
Perform ThinLTO on x86_64-pc-windows-msvc dist builder try-job: dist-x86_64-msvc
This comment has been minimized.
This comment has been minimized.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for 9f3fcfb failed: CI. Failed job:
|
This reenables ThinLTO on x86_64-pc-windows-msvc.
try-job: dist-x86_64-msvc