lgtm
http://codereview.chromium.org/10692058/diff/1/src/profile-generator.cc File src/profile-generator.cc (right): http://codereview.chromium.org/10692058/diff/1/src/profile-generator.cc#newcode2189 src/profile-generator.cc:2189: if (k->length() > 0) { Consider comparing against Heap::hidden_symbol() http://codereview.chromium.org/10692058/diff/1/src/profile-generator.cc#newcode2196 src/profile-generator.cc:2196: OS::Print("Empty name for fast property 1\n"); remove OS::Print http://codereview.chromium.org/10692058/diff/1/src/profile-generator.cc#newcode2205 src/profile-generator.cc:2205: if (k->length() > 0) { Consider comparing against Heap::hidden_symbol http://codereview.chromium.org/10692058/diff/1/src/profile-generator.cc#newcode2208 src/profile-generator.cc:2208: OS::Print("Empty name for fast property 2\n"); remove OS::Print http://codereview.chromium.org/10692058/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
