-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
ICE: rmeta/decoder.rs:1130: impossible case reached #71734
Comments
Updated the issue description with a complete backtrace that has full line info (435 frames deep!) |
@ecstatic-morse, it looks like this match needs to be updated somehow: rust/src/librustc_metadata/rmeta/decoder.rs Lines 1122 to 1132 in d20113d
Does this bug report contain enough information to understand the problem, or do you need an MCVE? |
Could you reproduce but debug print the the scrutinee in the |
I'm surprised since we should return before calling rust/src/librustc_mir_build/hair/pattern/mod.rs Lines 768 to 773 in d20113d
It must be one of the |
Indeed, here's the source for the item being checked (specifically, pub trait ServiceMarker: Sized + Send + Sync + 'static {
/// The type of the structure against which FIDL requests are made.
/// Queries made against the proxy are sent to the paired `ServerEnd`.
type Proxy: Proxy<Service = Self>;
/// The type of the stream of requests coming into a server.
type RequestStream: RequestStream<Service = Self>;
/// The name of the service suitable for debug purposes.
///
/// This will be removed-- users should switch to either
/// `DEBUG_NAME` or `DiscoverableService::NAME`.
const NAME: &'static str = Self::DEBUG_NAME;
/// The name of the service suitable for debug purposes.
///
/// For discoverable services, this should be identical to
/// `<Self as DiscoverableService>::NAME`.
const DEBUG_NAME: &'static str;
} |
Ah okay. We should already be encoding the |
No problem! Since I'd kicked this off already, here's the debug print:
|
This also tests for the ICE in rust-lang#71734
Decode qualifs for associated const defaults Fixes rust-lang#71734. We encode qualifs for associated constants, but never expected to decode the qualifs for defaulted associated consts. Fix this, and test that associated const defaults have the correct qualifs cross-crate. r? @tmandry
Code
I don't have a reasonable sized reproducer yet.
Meta
rustc --version --verbose
:Regression occurred
between bf1f2ee..fa51f81in 796c0ca (#67343)Error output
Backtrace
The text was updated successfully, but these errors were encountered: