Revision: 6502 Author: [email protected] Date: Wed Jan 26 10:15:43 2011 Log: Compress Variable class.
Review Link: http://codereview.chromium.org/6246019/ http://code.google.com/p/v8/source/detail?r=6502 Modified: /branches/bleeding_edge/src/variables.cc /branches/bleeding_edge/src/variables.h ======================================= --- /branches/bleeding_edge/src/variables.cc Mon Jan 17 00:11:03 2011 +++ /branches/bleeding_edge/src/variables.cc Wed Jan 26 10:15:43 2011 @@ -112,12 +112,12 @@ : scope_(scope), name_(name), mode_(mode), - is_valid_LHS_(is_valid_LHS), kind_(kind), local_if_not_shadowed_(NULL), + rewrite_(NULL), + is_valid_LHS_(is_valid_LHS), is_accessed_from_inner_scope_(false), - is_used_(false), - rewrite_(NULL) { + is_used_(false) { // names must be canonicalized for fast equality checks ASSERT(name->IsSymbol()); } ======================================= --- /branches/bleeding_edge/src/variables.h Mon Jan 17 00:11:03 2011 +++ /branches/bleeding_edge/src/variables.h Wed Jan 26 10:15:43 2011 @@ -187,21 +187,23 @@ Scope* scope_; Handle<String> name_; Mode mode_; - bool is_valid_LHS_; Kind kind_; Variable* local_if_not_shadowed_; - // Usage info. - bool is_accessed_from_inner_scope_; // set by variable resolver - bool is_used_; - // Static type information StaticType type_; // Code generation. // rewrite_ is usually a Slot or a Property, but may be any expression. Expression* rewrite_; + + // Valid as a LHS? (const and this are not valid LHS, for example) + bool is_valid_LHS_; + + // Usage info. + bool is_accessed_from_inner_scope_; // set by variable resolver + bool is_used_; }; -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
