tree 4fe6a934da07f24cf1f24c87890a10e558cfe5c6
parent 1efb130a8e5df86fc574dcf97cffc9a3390d838c
author Toon Verwaest <verwaest@chromium.org> 1547137349 +0100
committer Commit Bot <commit-bot@chromium.org> 1547207438 +0000

[parser] Move MarkLoopVariableAsAssigned to 'var' proxy allocation

This unifies the code between parser and preparser, and removes more code from
the pattern rewriter.

This makes "var x" without assignment and initializer in a loop pessimistically
marked as assigned, but that seems pretty unlikely since the variable will just
always be undefined. It is also still strictly better than what we had until
very recently since any var outside of the function scope used to be marked as
assigned. Now we only mark such variables as assigned.

Change-Id: Icb37ab249b2a79c2d57a5769bdb964b435cebf62
Reviewed-on: https://chromium-review.googlesource.com/c/1405228
Reviewed-by: Leszek Swirski <leszeks@chromium.org>
Commit-Queue: Toon Verwaest <verwaest@chromium.org>
Cr-Commit-Position: refs/heads/master@{#58736}