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

Missing optimized MIR for an item #126012

Closed
tmiasko opened this issue Jun 5, 2024 · 2 comments · Fixed by #125996
Closed

Missing optimized MIR for an item #126012

tmiasko opened this issue Jun 5, 2024 · 2 comments · Fixed by #125996
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ P-medium Medium priority regression-from-stable-to-stable Performance or correctness regression from one stable version to another.

Comments

@tmiasko
Copy link
Contributor

tmiasko commented Jun 5, 2024

$ cat a.rs 
pub static A: fn() -> fn() = {
    #[inline(never)]
    fn f() {}
    #[inline(always)]
    || -> fn() { f }
};
$ cat b.rs
static A: fn() -> fn () = a::A;
fn main() { A()() }
$ rustc a.rs --crate-type=lib
$ rustc b.rs --extern a -L.
error: missing optimized MIR for an item in the crate `a`
  |
note: missing optimized MIR for this item (was the crate `a` compiled with `--emit=metadata`?)
 --> /.../a.rs:3:5
  |
3 |     fn f() {}
  |     ^^^^^^

error: aborting due to 1 previous error

Regression in c563f2e (#122371 cc @oli-obk @tmiasko).

@rustbot modify labels: +regression-from-stable-to-stable -regression-untriaged

@tmiasko tmiasko added C-bug Category: This is a bug. regression-untriaged Untriaged performance or correctness regression. labels Jun 5, 2024
@tmiasko tmiasko self-assigned this Jun 5, 2024
@rustbot rustbot added I-prioritize Issue: Indicates that prioritization has been requested for this issue. needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. regression-from-stable-to-stable Performance or correctness regression from one stable version to another. and removed regression-untriaged Untriaged performance or correctness regression. labels Jun 5, 2024
@apiraino
Copy link
Contributor

apiraino commented Jun 5, 2024

WG-prioritization assigning priority (Zulip discussion).

@rustbot label -I-prioritize +P-medium

@rustbot rustbot added P-medium Medium priority and removed I-prioritize Issue: Indicates that prioritization has been requested for this issue. labels Jun 5, 2024
@bors bors closed this as completed in eb2819e Jun 5, 2024
rust-timer added a commit to rust-lang-ci/rust that referenced this issue Jun 5, 2024
Rollup merge of rust-lang#125996 - tmiasko:closure-recursively-reachable, r=oli-obk

Closures are recursively reachable

Fixes rust-lang#126012.
@tmiasko tmiasko removed their assignment Jun 5, 2024
@saethlin saethlin added I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Jun 6, 2024
@apiraino
Copy link
Contributor

apiraino commented Jun 6, 2024

Beta backport accepted as per compiler team on Zulip. A backport PR will be authored by the release team at the end of the current development cycle.

@rustbot label +beta-accepted

@rustbot rustbot added the beta-accepted Accepted for backporting to the compiler in the beta channel. label Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ P-medium Medium priority regression-from-stable-to-stable Performance or correctness regression from one stable version to another.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants