https://codereview.chromium.org/722793005/diff/1/src/ast.h
File src/ast.h (right):
https://codereview.chromium.org/722793005/diff/1/src/ast.h#newcode2675
src/ast.h:2675: VariableProxy* proxy() const { return proxy_; }
Nit: rename proxy => class_variable_proxy (also field)
"Proxy" is way overloaded.
https://codereview.chromium.org/722793005/diff/1/src/ast.h#newcode2684
src/ast.h:2684: VariableProxy* proxy, Expression* extends,
proxy => class_variable_proxy
https://codereview.chromium.org/722793005/diff/1/src/ast.h#newcode2699
src/ast.h:2699: VariableProxy* proxy_;
proxy_ => class_variable_proxy_
https://codereview.chromium.org/722793005/diff/1/src/ast.h#newcode3684
src/ast.h:3684: VariableProxy* proxy, Expression* extends,
proxy => class_variable_proxy
https://codereview.chromium.org/722793005/diff/1/src/full-codegen.cc
File src/full-codegen.cc (right):
https://codereview.chromium.org/722793005/diff/1/src/full-codegen.cc#newcode1613
src/full-codegen.cc:1613: Comment cmnt(masm_, "[ Extend block context");
Please share code with VisitBlock here (add an EnterBlockScope method)
We will also need bailout ids here in the future, when/if we
hydrogenize/turbofanize this.
I suggest extracting that part too (so both context pushing and visiting
declarations below are part of EnterBlockScope method), parameterizing
EnterBlockScope with BailoutIds and for now passing BailoutId::None
here.
https://codereview.chromium.org/722793005/diff/1/src/full-codegen.cc#newcode1651
src/full-codegen.cc:1651: LoadContextField(context_register(),
Context::PREVIOUS_INDEX);
Please share code with VisitBlock here (add an ExitBlockScope method)
Same as above for BailoutIds.
https://codereview.chromium.org/722793005/diff/1/src/parser.cc
File src/parser.cc (right):
https://codereview.chromium.org/722793005/diff/1/src/parser.cc#newcode3976
src/parser.cc:3976: if (has_seen_constructor !=
old_has_seen_constructor) {
Nit: replace this condition with 'constructor == NULL &&
has_seen_constructor' and remove old_has_seen_constructor.
https://codereview.chromium.org/722793005/diff/1/src/parser.cc#newcode3991
src/parser.cc:3991: if (!has_seen_constructor) {
Nit: replace this condition with 'constructor == NULL'
https://codereview.chromium.org/722793005/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
