LGTM (with one comment).
https://chromiumcodereview.appspot.com/10832157/diff/3001/src/ast.h File src/ast.h (right): https://chromiumcodereview.appspot.com/10832157/diff/3001/src/ast.h#newcode1813 src/ast.h:1813: BailoutId CountId() const { return count_id_; } I think we can get rid of this BailoutId accessor as well ... https://chromiumcodereview.appspot.com/10832157/diff/3001/src/ast.h#newcode1815 src/ast.h:1815: TypeFeedbackId CountBinOpFeedbackId() const { return reuse(CountId()); } ... and just use the underlying AST id directly here. https://chromiumcodereview.appspot.com/10832157/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
