Skip to content

[ddc] Generate temp names for the synthetic variables that are generated by the CFE #51589

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
Closed
annagrin opened this issue Mar 1, 2023 · 0 comments
Closed

[ddc] Generate temp names for the synthetic variables that are generated by the CFE #51589

annagrin opened this issue Mar 1, 2023 · 0 comments
Assignees
@annagrin
Labels
area-web-js Issues related to JavaScript support for Dart Web, including DDC, dart2js, and JS interop. dev-compiler-sdk P2 A bug or feature request we're likely to work on web-dev-compiler
Milestone
Dart 3 beta 3

Comments

@annagrin
Copy link
Contributor

annagrin commented Mar 1, 2023

Parent: dart-lang/webdev#1974

DWDS recognizes temp variables generated by DDC and does not show them in the debugger. Renaming the CFE variables to use the DDC temp scheme would help the debugger recognize and not display those are well.

Prerequisite: #51554
@annagrin annagrin added this to the Dart 3 beta 3 milestone Mar 1, 2023
@annagrin annagrin added the P2 A bug or feature request we're likely to work on label Mar 1, 2023
@annagrin annagrin mentioned this issue Mar 1, 2023
Closed
@lrhn lrhn added the area-web-js Issues related to JavaScript support for Dart Web, including DDC, dart2js, and JS interop. label Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@annagrin annagrin

Labels
area-web-js Issues related to JavaScript support for Dart Web, including DDC, dart2js, and JS interop. dev-compiler-sdk P2 A bug or feature request we're likely to work on web-dev-compiler
Projects
None yet
Milestone
Dart 3 beta 3
Development

No branches or pull requests
2 participants
@lrhn @annagrin