Okay, thanks! Landing...

https://codereview.chromium.org/1217943004/diff/20001/src/heap/heap.cc
File src/heap/heap.cc (right):

https://codereview.chromium.org/1217943004/diff/20001/src/heap/heap.cc#newcode4726
src/heap/heap.cc:4726: // object = EnsureDoubleAligned(this, object,
size);
On 2015/06/30 13:59:11, Hannes Payer wrote:
Do you have to double align the type feedback vector?

Nope. Removed comment, thx.

https://codereview.chromium.org/1217943004/diff/20001/src/type-feedback-vector.h
File src/type-feedback-vector.h (right):

https://codereview.chromium.org/1217943004/diff/20001/src/type-feedback-vector.h#newcode12
src/type-feedback-vector.h:12: // #include "src/heap/heap.h"
On 2015/06/30 09:43:43, Benedikt Meurer wrote:
Nit: Nuke unused includes. No need to leave them as comments.

Done.

https://codereview.chromium.org/1217943004/

--
--
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/d/optout.

Reply via email to