Thanks for the patch. LGTM, I will land this after I verify there is no performance regression.
http://codereview.chromium.org/11307012/diff/3001/src/heap.h File src/heap.h (right): http://codereview.chromium.org/11307012/diff/3001/src/heap.h#newcode1455 src/heap.h:1455: // they are in new space. I don't think this comment is correct. The comment about being in new space isn't accurate, and this patch doesn't check for it. I'll adjust before landing. http://codereview.chromium.org/11307012/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
