LGTM.
http://codereview.chromium.org/8506004/diff/1/src/profile-generator.cc File src/profile-generator.cc (right): http://codereview.chromium.org/8506004/diff/1/src/profile-generator.cc#newcode2123 src/profile-generator.cc:2123: // TODO(svenpanne): Should we really ignore accessors here? That's a good question. They are really a reference to a function or pair of functions, so we probably should not skip them. It also looks like they are counted in the dictionary case. I'll ping Mikhail about it. http://codereview.chromium.org/8506004/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
