-
Notifications
You must be signed in to change notification settings - Fork 28.9k
Add compile-expression service. #16161
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
tests? |
What is your plan with this PR? (No particular rush, I'm just cleaning out the review queue.) |
This is waiting for the dart sdk changes to land first, then this can be landed. |
b1d8b99
to
617d2fb
Compare
617d2fb
to
4d8c856
Compare
@aam Is there a bug or PR I can follow to see when the Dart SDK changes will land? |
Large chunk of dart sdk changes landed here: https://dart-review.googlesource.com/c/sdk/+/54407 I'm working on the rest of it, I don't have PR for it yet. |
Here is tracking bug for this: dart-lang/sdk#31981 |
Using cirrus successful run results as a signoff, the change has been sitting in appveyor's queue for 40 minutes already, hasn't started yet. |
This vm service is used during debugging to compile user-provided expressions into kernel format.