https://codereview.chromium.org/170703002/diff/1/src/heap.h
File src/heap.h (right):

https://codereview.chromium.org/170703002/diff/1/src/heap.h#newcode688
src/heap.h:688: // pretenure state.
What do you mean by "does not influence the pretenure state"?

https://codereview.chromium.org/170703002/diff/1/src/heap.h#newcode693
src/heap.h:693: AllocationSite* allocation_site = NULL);
What about implementing an AllocationMode, similar to HAllocationMode?

https://codereview.chromium.org/170703002/diff/1/src/heap.h#newcode693
src/heap.h:693: AllocationSite* allocation_site = NULL);
Why did you get rid of the Handle?

https://codereview.chromium.org/170703002/diff/1/src/heap.h#newcode774
src/heap.h:774: // points to the site. It doesn't influence pretenure
state.
same comments as above

https://codereview.chromium.org/170703002/diff/1/src/runtime.cc
File src/runtime.cc (right):

https://codereview.chromium.org/170703002/diff/1/src/runtime.cc#newcode14729
src/runtime.cc:14729: // don't emit a memento for them.
What about pretenuring in this case?

https://codereview.chromium.org/170703002/

--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to