Skip to content

Analyzer failure on generic function types with new phase ordering #31711

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.

Sign up for GitHub

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

Jump to bottom
Open
askeksa-google opened this issue Dec 22, 2017 · 3 comments
Open

Analyzer failure on generic function types with new phase ordering #31711

askeksa-google opened this issue Dec 22, 2017 · 3 comments
Labels
legacy-area-front-end Legacy: Use area-dart-model instead. P3 A lower priority bug or feature request

Comments

@askeksa-google
Copy link

https://dart-review.googlesource.com/c/sdk/+/22500 moves the finishTypeVariables phase earlier in order to have the type variable bounds in place when the program is built. This causes the resynthesis test test_typedef_generic_asFieldType to fail. Further investigation is needed.
@askeksa-google askeksa-google self-assigned this Dec 22, 2017
@askeksa-google
Copy link
Author

askeksa-google commented Dec 22, 2017 •
edited
Loading

OK, this turned out to be a duplicate of #31213 which wasn't actually fixed by the CL after all. It just causes a different crash now.

@askeksa-google
Copy link
Author

@stefantsov

@askeksa-google askeksa-google changed the title Analyzer failure on generic function types with new phase ordering Fasta crash on nested generic function types Dec 22, 2017
@floitschG floitschG added the legacy-area-front-end Legacy: Use area-dart-model instead. label Dec 22, 2017
@jensjoha jensjoha added the P2 A bug or feature request we're likely to work on label Jan 4, 2018
@askeksa-google askeksa-google added the crash Process exits with SIGSEGV, SIGABRT, etc. An unhandled exception is not a crash. label May 24, 2018
@askeksa-google askeksa-google added this to the Dart2Stable milestone May 24, 2018
@kmillikin kmillikin removed this from the Dart2Stable milestone May 30, 2018
@askeksa-google askeksa-google changed the title Fasta crash on nested generic function types Analyzer failure on generic function types with new phase ordering May 30, 2018
@askeksa-google
Copy link
Author

#31213 does not fail anymore, but the original failure for this issue is still present. So not a duplicate. Going back to the original meaning.

@askeksa-google askeksa-google added P3 A lower priority bug or feature request and removed P2 A bug or feature request we're likely to work on crash Process exits with SIGSEGV, SIGABRT, etc. An unhandled exception is not a crash. labels May 30, 2018
@askeksa-google askeksa-google removed their assignment May 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
legacy-area-front-end Legacy: Use area-dart-model instead. P3 A lower priority bug or feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@kmillikin @floitschG @jensjoha @askeksa-google