Use free variable analysis to improve activation performance
Created attachment 67715 [details] Patch
Comment on attachment 67715 [details] Patch 214 void useVariable(const Identifier* ident, bool isEval) Since there's only one caller to this function, and it just compares the property name to the global eval property name, I think you could remove the isEval flag and have this function compute it automatically. 220 void collectFreeVariables(Scope* oldScope, bool shouldTrackCapturedVariables) I think this would be clearer if you named "oldScope" "nestedScope" instead. Might be worth a comment defining what a "captured" variable is. r=me
Committed r67583: <http://trac.webkit.org/changeset/67583>