Skip to content

AOT compiler has few optimization for is and as checks that are not enabled for JIT #30632

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
mraleph opened this issue Sep 6, 2017 · 2 comments
Closed

AOT compiler has few optimization for is and as checks that are not enabled for JIT #30632

mraleph opened this issue Sep 6, 2017 · 2 comments
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable

Comments

@mraleph
Copy link
Member

mraleph commented Sep 6, 2017

We need to evaluate if they should just be ported to JIT environment if they are beneficial.
@mraleph mraleph added area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable labels Sep 6, 2017
@alexmarkov
Copy link
Contributor

Related issue: #30571

@mkustermann mkustermann self-assigned this Oct 9, 2018
dart-bot pushed a commit that referenced this issue Oct 10, 2018
@mkustermann
[VM] Use assert assignable implementation for explicit downcasts
16ddfc1 
This makes explicit downcasts more efficient by re-using the
optimizations we already have for implicit downcasts.

For implicit/explicit as checks we use the symbol to distinguish which
exception to throw if the check fails.  So this does not actually
require unification of _CastError and _TypeError.

Issue #34582
Issue #30632
Issue #30571

Change-Id: I7b8818d2698e249c90a3e1b8790bad69ada649a9
Reviewed-on: https://dart-review.googlesource.com/c/78748
Commit-Queue: Martin Kustermann <[email protected]>
Reviewed-by: Alexander Markov <[email protected]>
Reviewed-by: Vyacheslav Egorov <[email protected]>
@mkustermann mkustermann removed their assignment Sep 25, 2020
@mkustermann
Copy link
Member

After 1c7731d we use type testing stubs always - in JIT as well as AOT.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mraleph @mkustermann @alexmarkov