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
Crash when compiling big file #13235
Comments
Added Area-Dart2JS, Triaged labels. |
This comment was originally written by ngeoffray@google.com Set owner to ngeoffray@google.com. |
This comment was originally written by ngeoffray@google.com I mitigated the problem at revision r27471, but our code generation still uses a recusion algorithm that will be non-trivial to change, and provokes a stack overflow with your particular example. What you can do until we fix this problem (that in reality just occurs in tool generated Dart code), is to increase the stack size through your OS. Removed Priority-Unassigned label. |
This comment was originally written by kbec...@gmail.com Does splitting into multiple files or classes help in any way? |
This comment was originally written by ngeoffray@google.com It is the method size that is a problem here. The method TestUnit#run is too big for our code generator, so splitting it will definitely help. |
This comment was originally written by kbecker8...@gmail.com Ah ok, I can work around that.. Java has a much more severe restriction of 65k bytes in a method, so I have to take care of that there as well.. |
Added this to the M7 milestone. |
This comment was originally written by ngeoffray@google.com Added Triaged label. |
This comment was originally written by ngeoffray@google.com Set owner to @floitschG. |
Removed this from the Later milestone. |
Removed Oldschool-Milestone-Later label. |
The link above seems to be returning a 404 now, does anyone has a copy of the test file? |
This issue was originally filed by kbecke...@gmail.com
What steps will reproduce the problem?
What is the expected output? What do you see instead?
Expected no crash, so this:
The text was updated successfully, but these errors were encountered: