-
Notifications
You must be signed in to change notification settings - Fork 1.1k
3.7 regression: unhandled exception while running MegaPhase - map extension for (Int, Int) #22727
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
Comments
There is no crash when the code is changed from |
Could this be because of SIP62? #22652 |
II tested it and it doesn't seem to be connected to SIP-62, I removed the |
Last good nightly: 3.7.0-RC1-bin-20250222-4dc4668-NIGHTLY ~015f73fe686753e008cbf26a48fce457cd7fbf68 is the first bad commit. ~ cc @hamzaremmal EDIT: Further investigation indicates that the first bad commit is actually 7d79c56 (#22502) |
Euh, I don't see how my commit could have had an impact on this 😅 |
I looked at my linked follow-up PR at the duplicate ticket. It fixes the crash. I added the excellent minimization to the PR. |
Fixes scala#22705 Fixes scala#22706 Fixes scala#22727 Follow-up to scala#22502 by inserting a `dealias` when arriving at `target` type. Refactored the body of `hidden` to make it easier to read. Adjusted the doc for the same reason. As a reminder to self, the original reason for special handling of aliases was due to subclassing, but overrides are excluded. (One could restore that warning for edge cases.) The long doc explaining the handling of leading implicits is moved to the end (as an appendix). Despite best efforts, I was unable to make the doc longer than the code. [Cherry-picked d362492]
Fixes scala#22705 Fixes scala#22706 Fixes scala#22727 Follow-up to scala#22502 by inserting a `dealias` when arriving at `target` type. Refactored the body of `hidden` to make it easier to read. Adjusted the doc for the same reason. As a reminder to self, the original reason for special handling of aliases was due to subclassing, but overrides are excluded. (One could restore that warning for edge cases.) The long doc explaining the handling of leading implicits is moved to the end (as an appendix). Despite best efforts, I was unable to make the doc longer than the code. [Cherry-picked d362492]
Fixes scala#22705 Fixes scala#22706 Fixes scala#22727 Follow-up to scala#22502 by inserting a `dealias` when arriving at `target` type. Refactored the body of `hidden` to make it easier to read. Adjusted the doc for the same reason. As a reminder to self, the original reason for special handling of aliases was due to subclassing, but overrides are excluded. (One could restore that warning for edge cases.) The long doc explaining the handling of leading implicits is moved to the end (as an appendix). Despite best efforts, I was unable to make the doc longer than the code. [Cherry-picked d362492]
Compiler version
3.7.0-RC1-bin-20250228-e592b37-NIGHTLY
The same code works fine in 3.6.3 or 3.6.4-RC2.
Minimized code
Output (click arrow to expand)
The text was updated successfully, but these errors were encountered: