Stream: t-compiler/wg-prioritization/alerts

Topic: I-prioritize #75860 Compiler hang on function return type


triagebot (Aug 24 2020 at 00:04, on Zulip):

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

Procedure

Santiago Pastorino (Aug 26 2020 at 19:43, on Zulip):

bump, this is pending ...

Santiago Pastorino (Aug 26 2020 at 19:43, on Zulip):

sorry for bothering with these last messages, we've 4 more issues to prioritize

Santiago Pastorino (Aug 26 2020 at 19:43, on Zulip):

the ones on top :)

Santiago Pastorino (Aug 26 2020 at 19:45, on Zulip):

btw this one is weird ?

Santiago Pastorino (Aug 26 2020 at 19:46, on Zulip):

@WG-prioritization does someone have an idea why this could happen?

Joshua Nelson (Aug 26 2020 at 19:47, on Zulip):

the relevant snippet is 1,69% librustc_driver-74e90dce13cbc5f5.so [.] rustc_data_structures::obligation_forest::ObligationForest<O>::register_obligation_at

Joshua Nelson (Aug 26 2020 at 19:48, on Zulip):

so maybe some infinite recursion in typeck?

Santiago Pastorino (Aug 26 2020 at 19:49, on Zulip):

yeah, unsure

Santiago Pastorino (Sep 02 2020 at 14:49, on Zulip):

prioritizing this one is pending

Santiago Pastorino (Sep 02 2020 at 14:49, on Zulip):

but we don't have a way to reproduce this

Santiago Pastorino (Sep 02 2020 at 14:50, on Zulip):

I wonder if we should go with a priority and then adjust once someone is able to provide a repro for it

LeSeulArtichaut (Sep 02 2020 at 14:56, on Zulip):

Added E-needs-mcve, I think we should ping cleanup?

LeSeulArtichaut (Sep 02 2020 at 14:58, on Zulip):

I’d go with P-high, hangs are frustrating and this apparently has affected multiple people

Joshua Nelson (Sep 02 2020 at 14:59, on Zulip):

@LeSeulArtichaut it already has an MCVE

Joshua Nelson (Sep 02 2020 at 15:00, on Zulip):

the 'minimal POC' dropdown

Joshua Nelson (Sep 02 2020 at 15:01, on Zulip):

might want to edit the issue so it's not hidden by default

apiraino (Sep 02 2020 at 15:33, on Zulip):

Yeah I think a P-high is fine, can always be updated at a later stage. After a quick check on godbolt.org it seems to be broken since before 1.40, it's not clear since when this started :thinking: maybe a bisection could help?

Joshua Nelson (Sep 02 2020 at 15:34, on Zulip):

adding needs-bisection

Joshua Nelson (Sep 02 2020 at 15:36, on Zulip):

removed needs-mcve, it's already there

triagebot (Sep 02 2020 at 16:09, on Zulip):

Issue #75860's prioritization request has been removed.

Last update: Apr 11 2021 at 18:30UTC