more dbc:

https://chromiumcodereview.appspot.com/9221011/diff/4001/src/ast.h
File src/ast.h (right):

https://chromiumcodereview.appspot.com/9221011/diff/4001/src/ast.h#newcode1254
src/ast.h:1254: AstProperties* ast_properties_;
It seems quite expensive to add a pointer to each VariableProxy to
record later changes in the AstProperties. VariableProxy is one of the
most frequent of all AST nodes.

Maybe pass the AstProperties instead on to the phase that may change the
VariableProxy?

https://chromiumcodereview.appspot.com/9221011/diff/4001/src/objects.h
File src/objects.h (right):

https://chromiumcodereview.appspot.com/9221011/diff/4001/src/objects.h#newcode5412
src/objects.h:5412: kAstNodeCountOffset + kPointerSize;
You could reduce the x64 version by pairing two properties from above
like done for the ones below here.

https://chromiumcodereview.appspot.com/9221011/

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

Reply via email to