Committed patchset #11 manually as r17219 (presubmit successful).
https://codereview.chromium.org/24250005/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To u
Awesome, LGTM, just 2 nits
https://codereview.chromium.org/24250005/diff/54001/src/objects.cc
File src/objects.cc (right):
https://codereview.chromium.org/24250005/diff/54001/src/objects.cc#newcode5641
src/objects.cc:5641: // of the object if desired.
take advantage of 80 chars
https://coderev
Thx for the comments, here is my update.
https://codereview.chromium.org/24250005/diff/44001/src/allocation-site-scopes.h
File src/allocation-site-scopes.h (right):
https://codereview.chromium.org/24250005/diff/44001/src/allocation-site-scopes.h#newcode40
src/allocation-site-scopes.h:40: class
https://codereview.chromium.org/24250005/diff/44001/src/allocation-site-scopes.h
File src/allocation-site-scopes.h (right):
https://codereview.chromium.org/24250005/diff/44001/src/allocation-site-scopes.h#newcode40
src/allocation-site-scopes.h:40: class AllocationSiteContext {
Would it make sens
Hi Hannes, thx for the review, here is the update.
https://codereview.chromium.org/24250005/diff/27001/src/allocation-site-scopes.cc
File src/allocation-site-scopes.cc (right):
https://codereview.chromium.org/24250005/diff/27001/src/allocation-site-scopes.cc#newcode28
src/allocation-site-scopes
First round of comments, already looking awesome.
https://codereview.chromium.org/24250005/diff/27001/src/allocation-site-scopes.cc
File src/allocation-site-scopes.cc (right):
https://codereview.chromium.org/24250005/diff/27001/src/allocation-site-scopes.cc#newcode28
src/allocation-site-scopes.