Hmm ...

https://codereview.chromium.org/203523009/diff/30004/src/builtins.cc
File src/builtins.cc (right):

https://codereview.chromium.org/203523009/diff/30004/src/builtins.cc#newcode272
src/builtins.cc:272: heap->AdjustLiveBytes(elms->address(), -size_delta,
Heap::FROM_MUTATOR);
Unfortunately this is not correct. The adjustment happens after the mark
bits have been transferred but still targets the old address. Hence, the
mark bit is already gone.

Also I claim that AdjustLiveBytes() should not take an "Address" but and
"Object*" as a first parameter.

https://codereview.chromium.org/203523009/

--
--
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