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

#![feature(effects)] causes ICE #118628

Closed
peter-lyons-kehl opened this issue Dec 5, 2023 · 3 comments
Closed

#![feature(effects)] causes ICE #118628

peter-lyons-kehl opened this issue Dec 5, 2023 · 3 comments
Labels
C-bug Category: This is a bug. E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@peter-lyons-kehl
Copy link
Contributor

peter-lyons-kehl commented Dec 5, 2023

Long story: I'm rebasing my fork of Rust, mostly modified in library/alloc and library/core in 2023. I'm rebasing it on around 1 month old Rust master: 152a4e9. (I will re-base on the current master, I'll re-run and update here.)

My added/modified library/core code uses #[const_trait]. So I've added #![feature(effects)] to library/core/src/lib.rs. Then the attached ICE happens.

(The actual fork of Rust is out of scope here, but FYI https://github.com/coop-rs/rust/wiki https://github.com/coop-rs/rust - not on master, but being developed on https://github.com/coop-rs/rust/tree/07_vec_macroed_squash_fmt_workaround - but that is not the branch that triggers this).

UPDATE: My actual branch that triggers this ICE is: https://github.com/coop-rs/rust/tree/07_vec_macroed_squash_fmt_workaround_old_master_ice.

rustc-ice-2023-12-05T01:17:14.315936466Z-111587.txt

@peter-lyons-kehl peter-lyons-kehl added C-bug Category: This is a bug. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Dec 5, 2023
@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Dec 5, 2023
@aDotInTheVoid aDotInTheVoid added the E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example label Dec 5, 2023
@peter-lyons-kehl
Copy link
Contributor Author

UPDATE: My actual branch that triggers this ICE is: https://github.com/coop-rs/rust/tree/07_vec_macroed_squash_fmt_workaround_old_master_ice.

@peter-lyons-kehl
Copy link
Contributor Author

peter-lyons-kehl commented Dec 5, 2023

UPDATE: Still an ICE (I did not check if the exact same ICE or not) with whatever bootstrap comes on Dec 4, 2023, for this fork based on that same day's master da1da3f.

If I remove #![feature(effects)] from library/core/src/lib.rs, the ICE does go away (even though then there is a normal compiler error).

UPDATE: The fork code (being built) that still triggers the ICE with today's master is https://github.com/coop-rs/rust/tree/07_vec_macroed_squash_fmt_workaround_ICE_Dec-04-2023.

rustc-ice-2023-12-05T02_18_48-128485.txt

@fee1-dead
Copy link
Member

fee1-dead commented Dec 6, 2023

Dupe of #117004

@fee1-dead fee1-dead closed this as not planned Won't fix, can't repro, duplicate, stale Dec 6, 2023
@fmease fmease removed the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

5 participants