-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Compiler hangs under -Wunused with Scala 3.7.0-RC1 #22812
Comments
I'll take a look at
The workaround is turning off |
For information: currently I use in my project |
@OndrejSpanel Is it a public project that I can try? |
No, am sorry, it is not. It is quite complex project, ~500 Scala sources, proprietary sources. |
me too. complex project long scala sources You can try with compiler subproject commit https://github.com/chester-lang/chester/tree/5c90d7f86920459e80641465f48738844c8f5309 |
Confirmed the linked revert lets the example project proceed. |
This has to with linting inlined code; the revert just removes the tweak that recurses into inlined trees. |
The tweak to lint Inlined is reverted, on suspicion that it is the root cause of #22812 . (Investigation ongoing.) (If it reverts without a test change, then it doesn't have test coverage.) Gather legacy tests about type parameters and delete the `untried` test files. Same for `unused` test files; removing check and flags files for `warn-unused-privates` is a follow-up chore for a previous PR. Fixes #22812
Compiler version
3.7.0-RC1
3.7.0-RC1-bin-20250312-3fe9304-NIGHTLY
Minimized code
I do not have any and I assume producing it would be very time consuming, given the nature of the problem.
Output (click arrow to expand)
When using recent 3.7 compiler on my project, the compiler is stuck forever, profiler shows function like
dotty.tools.dotc.transform.MegaPhase.transformTree(Trees$Tree, int, Contexts$Context)
to be called with deep nesting.This must be a very recent regression. The same code compiles with 3.7.0-RC1-bin-20250309-2f639e2-NIGHTLY
The text was updated successfully, but these errors were encountered: