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

Remove benches from rustc_middle #102270

Merged
merged 1 commit into from
Sep 26, 2022

Conversation

Noratrieb
Copy link
Member

These benches benchmark rust langauge features and not the compiler, so they seem to be in the wrong place here. They also all take <1ns, making them pretty useless. Looking at their git history, they just seem to have been carried around for many, many years. This commit ends their journey.

These benches benchmark rust langauge features and not the compiler,
so they seem to be in the wrong place here. They also all take <1ns,
making them pretty useless. Looking at their git history, they just
seem to have been carried around for many, many years. This commit
ends their journey.
@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Sep 25, 2022
@rust-highfive
Copy link
Collaborator

r? @TaKO8Ki

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 25, 2022
@TaKO8Ki
Copy link
Member

TaKO8Ki commented Sep 26, 2022

Thank you! @bors r+ rollup

@bors
Copy link
Contributor

bors commented Sep 26, 2022

📌 Commit ddf7982 has been approved by TaKO8Ki

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 26, 2022
bors added a commit to rust-lang-ci/rust that referenced this pull request Sep 26, 2022
…fee1-dead

Rollup of 5 pull requests

Successful merges:

 - rust-lang#102143 (Recover from struct nested in struct)
 - rust-lang#102178 (bootstrap: the backtrace feature is stable, no need to allow it any more)
 - rust-lang#102197 (Stabilize const `BTree{Map,Set}::new`)
 - rust-lang#102267 (Don't set RUSTC in the bootstrap build script)
 - rust-lang#102270 (Remove benches from `rustc_middle`)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 39c6bdc into rust-lang:master Sep 26, 2022
@rustbot rustbot added this to the 1.66.0 milestone Sep 26, 2022
@Noratrieb Noratrieb deleted the delete-useless-benches branch September 26, 2022 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants