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

Revert "Auto merge of #91354 - fee1-dead:const_env, r=spastorino" #91491

Merged
merged 1 commit into from
Dec 3, 2021

Conversation

spastorino
Copy link
Member

This reverts commit 18bb8c6, reversing
changes made to d9baa36.

Reverts #91354 in order to address #91489. We would need to place this changes in a more granular way and would also be nice to address the small perf regression that was also introduced.

r? @oli-obk
cc @fee1-dead

…rino"

This reverts commit 18bb8c6, reversing
changes made to d9baa36.
@rust-highfive
Copy link
Collaborator

Some changes occurred in src/tools/clippy.

cc @rust-lang/clippy

Some changes occured to the CTFE / Miri engine

cc @rust-lang/miri

Some changes occured to the CTFE / Miri engine

cc @rust-lang/miri

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 3, 2021
@oli-obk
Copy link
Contributor

oli-obk commented Dec 3, 2021

@bors r+ rollup=never p=1

@bors
Copy link
Contributor

bors commented Dec 3, 2021

📌 Commit 85b723c has been approved by oli-obk

@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 Dec 3, 2021
@bors
Copy link
Contributor

bors commented Dec 3, 2021

⌛ Testing commit 85b723c with merge ff2439b...

@bors
Copy link
Contributor

bors commented Dec 3, 2021

☀️ Test successful - checks-actions
Approved by: oli-obk
Pushing ff2439b to master...

@rust-timer
Copy link
Collaborator

Finished benchmarking commit (ff2439b): comparison url.

Summary: This change led to moderate relevant improvements 🎉 in compiler performance.

  • Moderate improvement in instruction counts (up to -1.1% on incr-patched: b9b3e592dd cherry picked builds of style-servo)

If you disagree with this performance assessment, please file an issue in rust-lang/rustc-perf.

@rustbot label: -perf-regression

fee1-dead added a commit to fee1-dead-contrib/rust that referenced this pull request Dec 12, 2021
…i-obk"

This reverts commit ff2439b, reversing
changes made to 2a9e083.
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 12, 2021
Eliminate ConstnessAnd again

Closes rust-lang#91489.
Closes rust-lang#89432.

Reverts rust-lang#91491.
Reverts rust-lang#89450.

r? `@spastorino`
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 17, 2021
…i-obk"

This reverts commit ff2439b, reversing
changes made to 2a9e083.
flip1995 pushed a commit to flip1995/rust that referenced this pull request Dec 17, 2021
Eliminate ConstnessAnd again

Closes rust-lang#91489.
Closes rust-lang#89432.

Reverts rust-lang#91491.
Reverts rust-lang#89450.

r? `@spastorino`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants