Repository navigation
bootstrap: keep intrinsic-test path compatibility - #164101
HNO3Miracle wants to merge 1 commit into
Conversation
|
|
|
Hi, what exactly do you mean by "external CI jobs"? We specifically made this change because we don't want |
|
Thanks for pointing this out. By “external CI jobs”, I mean the ISCAS/Ruyi RISC-V CI: https://community-ci.openruyi.cn/ruyici/job/RustCI/job/Rustci-daily-job/ That CI currently invokes: ./x.py test --stage 2 library/stdarch/crates/intrinsic-test After #159898, this fails during bootstrap path resolution because the step only registers the The failure is visible here: I agree that registering the path makes I will follow your preference here. |
Fixes a failure in the ISCAS/Ruyi RISC-V CI:
https://community-ci.openruyi.cn/ruyici/job/RustCI/job/Rustci-daily-job/
The CI currently invokes:
Since
IntrinsicTest::should_runwas changed to register only theintrinsic-testalias, this invocation fails during bootstrap path resolutionwith:
The failure is visible in the CI artifact for run 38021940247:
https://community-ci.openruyi.cn/misc/rustci/gate-ci/2026-10-10/baff6033863e38fe5f57242ea99b7f960ce9863c/failure-case.log
This PR restores the historical path as an additional selector. The existing
architecture check still skips intrinsic-test on unsupported hosts such as
RISC-V.
However, I understand that registering the path also makes
x test libraryselect this step. If that behavior is intentionally forbidden, the better fix
is to update the external CI invocation to use
intrinsic-testinstead, andthis PR should be closed rather than changing bootstrap path matching.
Tests:
./x.py test bootstrapr? @Kobzol