[Webkit-unassigned] [Bug 74633] Allow JSStaticScope to bind more than one variable

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=74633





--- Comment #8 from Gavin Barraclough <barraclough at apple.com>  2011-12-25 15:38:18 PST ---
<Gah, accidental commit!>

... , and having differing mechanisms underlying const/let/var may complicate implementation (e.g. activation based lazy tear-off results in eager virtual register allocation, use of static scopes does not – in the case of activation, if capture analysis shows a variable cannot be captured the only change in code generation might be to omit the create-activation/tear-off checks, at the points variables are accessed there would be no change in the byte code emitted).

In the case of optimizations to const/let that are additive to the basic mechanism, there is no need for these to be considered as part of the initial implementation (indeed I'd argue it's best not to – to land the code in its simplest functional, correct form, and to optimize incrementally from there showing the benefit of each change).  But we don't want to land large changed that would only be torn out weeks later, so I think the question of whether the const/let implementation should be based on static scope objects or activations is a fundamental question that really ought to be addressed before proceeding.

What do you think?

cheers,
G.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.