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: associated consts cannot be referenced in patterns #103242

Closed
Mark-Simulacrum opened this issue Oct 19, 2022 · 5 comments
Closed

regression: associated consts cannot be referenced in patterns #103242

Mark-Simulacrum opened this issue Oct 19, 2022 · 5 comments
Assignees
Labels
E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example P-critical Critical priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Milestone

Comments

@Mark-Simulacrum
Copy link
Member

Really bad spans in all of these (not related to this bug, I expect), but still a pretty wide ranging regression.

@Mark-Simulacrum Mark-Simulacrum added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. 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 E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example labels Oct 19, 2022
@Mark-Simulacrum Mark-Simulacrum added this to the 1.65.0 milestone Oct 19, 2022
@rustbot rustbot added the I-prioritize Issue: Indicates that prioritization has been requested for this issue. label Oct 19, 2022
@boulanlo
Copy link

Hi! A bit new to Rust contributing, but I saw this got a "needs bisection" label and thought I'd try to help. I picked the crate from the last log in the list (zkink).

I've managed to run the cargo-bisect-rustc tool up to the actual nightly that regressed (nightly-2022-09-09), but couldn't go further due to weird download issues with the actual commits.

Here's the list of relevant commits the tool has found:

found 9 bors merge commits in the specified range
  commit[0] 2022-09-07UTC: Auto merge of #101544 - matthiaskrgr:rollup-4urx917, r=matthiaskrgr
  commit[1] 2022-09-07UTC: Auto merge of #94075 - mikebenfield:wip-enum, r=oli-obk
  commit[2] 2022-09-08UTC: Auto merge of #101303 - jyn514:jnelson/handle-cycle-enum, r=cjgillot
  commit[3] 2022-09-08UTC: Auto merge of #101504 - lqd:rust-lld-fix, r=petrochenkov
  commit[4] 2022-09-08UTC: Auto merge of #101560 - Dylan-DPC:rollup-8m8yr89, r=Dylan-DPC
  commit[5] 2022-09-08UTC: Auto merge of #98900 - lcnr:region-stuff, r=jackh726
  commit[6] 2022-09-08UTC: Auto merge of #101467 - nnethercote:shrink-hir-Ty-Pat, r=spastorino
  commit[7] 2022-09-08UTC: Auto merge of #101577 - Dylan-DPC:rollup-l9xw7i7, r=Dylan-DPC
  commit[8] 2022-09-08UTC: Auto merge of #101437 - compiler-errors:erase-normalize-ordering, r=tmandry

Out of all of these, I think #98900 looks like the culprit the most, seeing it has something to do with associated consts. I may be wrong though, and I didn't investigate the actual error.

(I hope that I bisected correctly; since I am on NixOS, I've ran into some trouble getting patched executables when bisecting, and since I'm seeing a PR about lld in the commit list, I'm afraid I messed up somehow!)

@Mark-Simulacrum
Copy link
Member Author

Cc @lcnr as author of #98900

@Mark-Simulacrum Mark-Simulacrum removed the E-needs-bisection Call for participation: This issue needs bisection: https://github.com/rust-lang/cargo-bisect-rustc label Oct 19, 2022
@compiler-errors
Copy link
Member

compiler-errors commented Oct 20, 2022

This is also fixed by #103279.

@apiraino
Copy link
Contributor

WG-prioritization assigning priority (Zulip discussion).

@rustbot label -I-prioritize +P-critical

@rustbot rustbot added P-critical Critical priority and removed I-prioritize Issue: Indicates that prioritization has been requested for this issue. labels Oct 20, 2022
@compiler-errors compiler-errors self-assigned this Oct 20, 2022
@compiler-errors
Copy link
Member

1.65 is released

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
E-needs-mcve Call for participation: This issue has a repro, but needs a Minimal Complete and Verifiable Example P-critical Critical priority regression-from-stable-to-beta Performance or correctness regression from stable to beta. 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