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.

Sign up for GitHub

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

Jump to bottom

Wire up testing to ./x.py test in rust-lang/rust #1290

Open
10 of 13 tasks
bjorn3 opened this issue Oct 12, 2022 · 3 comments
Open
10 of 13 tasks

Wire up testing to ./x.py test in rust-lang/rust #1290

bjorn3 opened this issue Oct 12, 2022 · 3 comments
Labels
A-testing Area: Testing of cg_clif itself C-enhancement Category: An issue proposing an enhancement or a PR with one.
Milestone
Distribute as rus...

Comments

@bjorn3
Copy link
Member

bjorn3 commented Oct 12, 2022 •
edited
Loading

@bjorn3 bjorn3 added C-enhancement Category: An issue proposing an enhancement or a PR with one. A-testing Area: Testing of cg_clif itself labels Oct 12, 2022
@bjorn3 bjorn3 added this to the Distribute as rustup component milestone Oct 12, 2022
This was referenced Jan 5, 2023
Merged
Closed
bjorn3 added a commit that referenced this issue Jan 11, 2023
@bjorn3
Don't install hyperfine in y.rs prepare
fb788e3 
Missed in #1338
Part of #1290
bjorn3 added a commit that referenced this issue Jan 13, 2023
@bjorn3
Move LLVM simple-raytracer build to ./y.rs bench
dbfbb29 
cc #1290
This was referenced Jan 13, 2023
Merged
Merged
Merged
Closed
@jyn514
Copy link
Member

jyn514 commented Jan 17, 2023 •
edited
Loading

Support building with specified rustc and cargo rather than rustup managed one

If you respect RUSTC and CARGO, I think bootstrap already sets those today :) I wouldn't expect you to need a new argument to y.rs or anything like that.

https://github.com/bjorn3/rustc_codegen_cranelift/issues/1282

cc https://rust-lang.zulipchat.com/#narrow/stream/238009-t-compiler.2Fmeetings/topic/.5Bweekly.5D.202023-01-05/near/319616098 - I think this is ok not to require for now :)

@bjorn3 bjorn3 mentioned this issue Feb 13, 2023
Merged
@bjorn3 bjorn3 mentioned this issue Jun 16, 2023
Merged
@bjorn3
Copy link
Member Author

bjorn3 commented Jun 16, 2023

Opened rust-lang/rust#112701

bors referenced this issue in rust-lang-ci/rust Sep 10, 2023
@bors
Auto merge of rust-lang#112701 - bjorn3:test_cg_clif_in_ci2, r=Mark-S…
c0583a0 
…imulacrum

Run part of cg_clif's tests in CI

While currently cg_clif is already built in CI ensuring that it always compiles, sometimes there is a bug in the changes that were made causing tests to fail. This PR is one step towards preventing this.

Part of the test suite is still skipped until vendoring for the projects that are being tested is implemented. I will implement that in a future PR.

Fixes part of https://github.com/bjorn3/rustc_codegen_cranelift/issues/1290
Fixes part of rust-lang#95518
Required for rust-lang#81746
@bjorn3
Copy link
Member Author

bjorn3 commented Oct 10, 2023 •
edited
Loading

The no_sysroot and base_sysroot parts of cg_clif's test suite has been running on rust's CI now for almost a month. The extended_sysroot part is blocked on vendoring support of the things being tested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-testing Area: Testing of cg_clif itself C-enhancement Category: An issue proposing an enhancement or a PR with one.
Projects
None yet
Milestone
Distribute as rustup component
Development

No branches or pull requests
2 participants
@bjorn3 @jyn514