LGTM
http://codereview.chromium.org/5139002/diff/1/src/profile-generator.cc File src/profile-generator.cc (right): http://codereview.chromium.org/5139002/diff/1/src/profile-generator.cc#newcode1476 src/profile-generator.cc:1476: const uint64_t HeapObjectsMap::kGcRootsObjectId = 1; Please add kNumReservedObjectsIds = 2 (maybe you can find a better name) and use instead of 2 below. http://codereview.chromium.org/5139002/diff/1/src/profile-generator.cc#newcode1863 src/profile-generator.cc:1863: Add empty line. http://codereview.chromium.org/5139002/diff/1/src/profile-generator.cc#newcode1914 src/profile-generator.cc:1914: HeapObjectsSet* knownReferences = NULL) knownReferences -> known_references http://codereview.chromium.org/5139002/diff/1/src/profile-generator.h File src/profile-generator.h (right): http://codereview.chromium.org/5139002/diff/1/src/profile-generator.h#newcode703 src/profile-generator.h:703: static HeapObject *const kInternalRootObject; Move * closer to HeapObject. http://codereview.chromium.org/5139002/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
