tree dd5991eab264f83ba6e22e1032ec556d37de3acf
parent 2e3ac0cf154ab0387fc6b8859480a0a27a67fcb3
author Johnni Winther <johnniwinther@google.com> 1612599076 +0000
committer commit-bot@chromium.org <commit-bot@chromium.org> 1612599076 +0000

Reland: [dart2js] Refactor closure data to use AST nodes instead of locals

This CL avoids creating locals using KernelToLocalsMap when computing
closure data for the J-model. Instead the closure data uses
ir.VariableDeclaration, JTypeVariable, and internal locals (ThisLocal
and BoxLocal) internally, and the accessor function require passes
a KernelToLocalsMap to convert the internal model to a Locals upon use.
The converted Locals are cached internal for performance only.

This prepares for splitting the GlobalLocalsMap and KernelToLocalsMap
from the JClosedWorld.

Previously landed in https://dart-review.googlesource.com/c/sdk/+/180820
but reverted.

Change-Id: Ic6a2ab63de7cb31c89dd50ca2af3f205a792fd74
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/183000
Reviewed-by: Sigmund Cherem <sigmund@google.com>