Skip to content

[DDC] Failures in end-to-end expression compiler tests #51825

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

Closed
fishythefish opened this issue Mar 22, 2023 · 1 comment
Closed

[DDC] Failures in end-to-end expression compiler tests #51825

fishythefish opened this issue Mar 22, 2023 · 1 comment
Assignees
Labels
area-web-js Issues related to JavaScript support for Dart Web, including DDC, dart2js, and JS interop. dev-compiler-eval type-test web-eval evaluation support on the web
Milestone

Comments

@fishythefish
Copy link
Member

pkg/dev_compiler/test/expression_compiler/expression_compiler_e2e_ddc_unsound_test is failing on unittest-asserts-debug-linux and unittest-asserts-release-mac-arm64. (It went from flaky to timeout on the former rather than failing with a runtime error.)

pkg/dev_compiler/test/expression_compiler/expression_compiler_e2e_amd_sound_test, pkg/dev_compiler/test/expression_compiler/expression_compiler_e2e_amd_unsound_test, and pkg/dev_compiler/test/expression_compiler/expression_compiler_e2e_ddc_sound_test are all failing on unittest-asserts-release-mac and unittest-asserts-release-mac-arm64.

@fishythefish fishythefish added area-web-js Issues related to JavaScript support for Dart Web, including DDC, dart2js, and JS interop. type-test web-eval evaluation support on the web labels Mar 22, 2023
@annagrin
Copy link
Contributor

annagrin commented Mar 23, 2023

Thanks @fishythefish! There are couple of issues causing the failures:

  • CFE change to improve the switch statements for patterns breaks incorrect test expectations in expression compiler e2e tests.
    04e6f4c

  • There seems to be an issue detecting the end of a scope for blocks inside switch statements due to synthesized variables that appear before user block but have file offsets from later code.

Fix: https://dart-review.googlesource.com/c/sdk/+/290706

@annagrin annagrin added this to the Dart 3 beta 3 milestone Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-web-js Issues related to JavaScript support for Dart Web, including DDC, dart2js, and JS interop. dev-compiler-eval type-test web-eval evaluation support on the web
Projects
None yet
Development

No branches or pull requests

2 participants