You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently we're just setting late fields to nullable in the type propagator, because otherwise all the sentinel comparison logic gets optimised out. This works, but is not optimal.
We'll need to extend CompileType to be able to handle lateness.
The text was updated successfully, but these errors were encountered:
liamappelbe
added
area-vm
Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
NNBD
Issues related to NNBD Release
labels
Feb 26, 2020
I don't think you should extend CompileType for this purpose. I suggest instead look at unifying late field lowering and InitStaticField and then either write a specialized data flow pass which removes redundant Init* instructions (if we want to handle complicated cases) or just permit CSE to remove Init* by dominance.
Currently we're just setting late fields to nullable in the type propagator, because otherwise all the sentinel comparison logic gets optimised out. This works, but is not optimal.
We'll need to extend CompileType to be able to handle lateness.
The text was updated successfully, but these errors were encountered: