LGTM.

http://codereview.chromium.org/546075/diff/1/5
File src/compiler.cc (right):

http://codereview.chromium.org/546075/diff/1/5#newcode757
src/compiler.cc:757: if (!var->is_global()) {
This is clearer than before and should be fine.

It just seems confusing that VariableProxy::AsVariable() checks var_ for
NULL explicitly. Is there any case where it  still can be NULL?

http://codereview.chromium.org/546075
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to