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

MaybeStorage improvements #133475

Merged
merged 4 commits into from
Nov 27, 2024
Merged

Conversation

nnethercote
Copy link
Contributor

Minor dataflow improvements.

r? @tmiasko

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 25, 2024
@rustbot
Copy link
Collaborator

rustbot commented Nov 25, 2024

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

Some changes occurred to the CTFE machinery

cc @rust-lang/wg-const-eval

Some changes occurred to the CTFE / Miri interpreter

cc @rust-lang/miri

@rust-log-analyzer

This comment has been minimized.

@nnethercote nnethercote force-pushed the MaybeStorage-improvements branch from af639f7 to aa73b97 Compare November 26, 2024 00:42
@rust-log-analyzer

This comment has been minimized.

We can union the two sets the easy way. This removes the need for the
domain size check, because `union` does that same check itself.
It's very closely related to `MaybeStorageLive` and `MaybeStorageDead`.
It's weird that it's currently in a different module.
- drop_flag_effects: `pub` items within are all re-exported in `lib.rs`.
- un_derefer: doesn't contain any `pub` items.
@nnethercote nnethercote force-pushed the MaybeStorage-improvements branch from aa73b97 to a602cb6 Compare November 26, 2024 01:11
@lcnr
Copy link
Contributor

lcnr commented Nov 26, 2024

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Nov 26, 2024

📌 Commit a602cb6 has been approved by lcnr

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 Nov 26, 2024
@lcnr
Copy link
Contributor

lcnr commented Nov 26, 2024

r? @lcnr

@rustbot rustbot assigned lcnr and unassigned tmiasko Nov 26, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Nov 27, 2024
…mpiler-errors

Rollup of 8 pull requests

Successful merges:

 - rust-lang#115293 (Remove -Zfuel.)
 - rust-lang#132605 (CI: increase timeout from 4h to 6h)
 - rust-lang#133304 (Revert diagnostics hack to fix ICE 132920)
 - rust-lang#133402 (Constify `Drop` and `Destruct`)
 - rust-lang#133458 (Fix `Result` and `Option` not getting a jump to def link generated)
 - rust-lang#133471 (gce: fix typing_mode mismatch)
 - rust-lang#133475 (`MaybeStorage` improvements)
 - rust-lang#133513 (Only ignore windows-gnu in avr-jmp-offset)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 219b2a0 into rust-lang:master Nov 27, 2024
6 checks passed
@rustbot rustbot added this to the 1.85.0 milestone Nov 27, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Nov 27, 2024
Rollup merge of rust-lang#133475 - nnethercote:MaybeStorage-improvements, r=lcnr

`MaybeStorage` improvements

Minor dataflow improvements.

r? `@tmiasko`
@nnethercote nnethercote deleted the MaybeStorage-improvements branch November 27, 2024 10:47
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.

6 participants