-
Notifications
You must be signed in to change notification settings - Fork 1.1k
crash with StackOverflowError #1661
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
Thanks for reporting @helloqirun . We touched this issue before (#1257), compile with the option
We'll look if we can improve the code to avoid stackoverflow in this simple case. |
Yes, we know that some code can blow the stack size, typically in Erasure. Two things:
|
I'd say that a systematic fix would be to make erasure use less stack. For example by making it be less-of-a typer, hopefully more of a mini-phase. |
Fixed on master. |
Yes, we should add the test case as a regression test. It is related to #2903, but without the added complexity of extracting parameters. |
This case is a bit large. I tried to minimize it, without much luck.
$ cat abc.scala
$ dotc abc.scala
The text was updated successfully, but these errors were encountered: