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

regression: binop(?) type inference failures #91872

Closed
Mark-Simulacrum opened this issue Dec 13, 2021 · 8 comments
Closed

regression: binop(?) type inference failures #91872

Mark-Simulacrum opened this issue Dec 13, 2021 · 8 comments
Labels
E-needs-bisection Call for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustc regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Milestone

Comments

@Mark-Simulacrum
Copy link
Member

A bunch of errors around type inference failure detected in the crater run -- I would like us to track down the PR causing these since I'm not finding an obvious reason off hand:

It seems plausible this is the breakage noted in #90904, but we should confirm.

@Mark-Simulacrum Mark-Simulacrum added regression-from-stable-to-beta Performance or correctness regression from stable to beta. E-needs-bisection Call for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustc labels Dec 13, 2021
@Mark-Simulacrum Mark-Simulacrum added this to the 1.58.0 milestone Dec 13, 2021
@rustbot rustbot added the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Dec 13, 2021
@Mark-Simulacrum Mark-Simulacrum added the T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. label Dec 15, 2021
@Mark-Simulacrum Mark-Simulacrum removed the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Dec 15, 2021
@Mark-Simulacrum
Copy link
Member Author

We will want to do a re-run of beta crater once #91484 or equivalent is beta-backported to confirm the majority/all of these are fixed before closing this, but generally it's likely that they will be.

@BGR360
Copy link
Contributor

BGR360 commented Dec 21, 2021

I just did some bisection the other day; it's pretty painless on my M1 pro macbook (~2-4 minutes per build). If this still needs bisection, and somebody can give me a range of commits, I'd be happy to try to bisect it.

@workingjubilee
Copy link
Member

Since each regression is likely due to #89167,
and each is likely fixed after #91484
we want to check, for each regression:

And any curiosities can be handled from there.

@BGR360
Copy link
Contributor

BGR360 commented Dec 22, 2021

For the regressions starting with reg/, where do I get those? The other ones have git repos I can clone (gh/), but I don't intuit what "reg" stands for.

EDIT: oh, does it just mean reg-ular crates.io crates?

@BGR360
Copy link
Contributor

BGR360 commented Dec 22, 2021

Here's what I got with the GitHub ones.

e90c5fb:

==== SUCCESSFULLY BUILT: ====
DaTa-.advent-of-code-2020
GCYYfun.OSTEP_RUST
MikkelPaulson.advent-of-code-2019
bfollek.advent2018rust
botwob.AdventOfCode2019
daniellockyer.wrp
l0s.advent-of-code-rust
mwc10.dt-backup
proelbtn.chacha20poly1305
qinxiaoguang.rs-lc
ruudjelinssen.aoc2020
==== FAILURES: ====
desbma.motd

3e018ce (one merge after 032dfe4):

==== SUCCESSFULLY BUILT: ====
==== FAILURES: ====
DaTa-.advent-of-code-2020
GCYYfun.OSTEP_RUST
MikkelPaulson.advent-of-code-2019
bfollek.advent2018rust
botwob.AdventOfCode2019
daniellockyer.wrp
desbma.motd
l0s.advent-of-code-rust
mwc10.dt-backup
proelbtn.chacha20poly1305
qinxiaoguang.rs-lc
ruudjelinssen.aoc2020

11fb21f

==== SUCCESSFULLY BUILT: ====
DaTa-.advent-of-code-2020
GCYYfun.OSTEP_RUST
MikkelPaulson.advent-of-code-2019
bfollek.advent2018rust
botwob.AdventOfCode2019
daniellockyer.wrp
l0s.advent-of-code-rust
mwc10.dt-backup
proelbtn.chacha20poly1305
qinxiaoguang.rs-lc
ruudjelinssen.aoc2020
==== FAILURES: ====
desbma.motd

Not sure what's up with desbma.motd. I made sure to check out the same revision of the repo that was used in the crater logs, but it failed to compile even on stable (with a different error than that in the crater logs). Maybe the author force-pushed that commit or something.

@workingjubilee
Copy link
Member

Thank you, it looks like it was in fact that PR then and that did in fact fix it, it seems.
Ah, I suppose we could also check 477fd70, but I will likely do that when I try to reimplement splatting.

@pietroalbini
Copy link
Member

Hmm, what's the status of this?

@Mark-Simulacrum
Copy link
Member Author

Should be fixed by #92592 on beta, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-needs-bisection Call for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustc regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

5 participants