Skip to content

Remove benches from rustc_middle#102270

Merged
bors merged 1 commit into
rust-lang:masterfrom
Noratrieb:delete-useless-benches
Sep 26, 2022
Merged

Remove benches from rustc_middle#102270
bors merged 1 commit into
rust-lang:masterfrom
Noratrieb:delete-useless-benches

Conversation

@Noratrieb

Copy link
Copy Markdown
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
Copy Markdown
Contributor

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

TaKO8Ki commented Sep 26, 2022

Copy link
Copy Markdown
Member

Thank you! @bors r+ rollup

@bors

bors commented Sep 26, 2022

Copy link
Copy Markdown
Collaborator

📌 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