Stream: t-compiler/wg-prioritization/alerts

Topic: I-prioritize #77993 ICE: tuple_fields called on non-tuple:…

triagebot (Oct 16 2020 at 18:43, on Zulip):

@WG-prioritization/alerts issue #77993 has been requested for prioritization.

Procedure

Camelid (Oct 16 2020 at 18:45, on Zulip):

This code was incorrect as it is, so error -> ICE. However, it seems like there's something weird going on here, so I suggest P-high until we get more information

Stu (Oct 16 2020 at 18:47, on Zulip):

It seems like macro can be replaced with any other keyword.

async fn ice() -> Box<dyn std::error::Error> {
    dyn!()
}

will panic too

Stu (Oct 16 2020 at 18:49, on Zulip):

I would go with P-high because it will panic if any keyword is followed by an unexpected token inside an async function that returns Box<dyn Trait>. This seems very weird

Hameer Abbasi (Oct 16 2020 at 18:50, on Zulip):

I suggest P-high, this pattern hits production code according to https://github.com/rust-lang/rust/issues/77993#issuecomment-709631876.

Stu (Oct 16 2020 at 18:51, on Zulip):

actix-web fails to build because of this

Hameer Abbasi (Oct 16 2020 at 18:55, on Zulip):

Then... P-critical?

Yuki Okushi (Oct 16 2020 at 18:56, on Zulip):

Stu said:

actix-web fails to build because of this

note that it occurs on the diagnostics test which is expected to fail :)

Camelid (Oct 16 2020 at 18:56, on Zulip):

Yes, this code is incorrect IIUC

Camelid (Oct 16 2020 at 18:56, on Zulip):

I'll assign P-high

triagebot (Oct 16 2020 at 18:57, on Zulip):

Issue #77993's prioritization request has been removed.

Last update: Apr 15 2021 at 01:15UTC