[v8-dev] Re: [heap] Move IdentityMap data structure out of heap. (issue 1320503004 by mstarzin...@chromium.org)

2015-08-28 Thread rmcilroy
lgtm, thanks! https://codereview.chromium.org/1320503004/ -- -- 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 unsubscribe from this group and stop receiving emai

[v8-dev] Re: [heap] Move IdentityMap data structure out of heap. (issue 1320503004 by mstarzin...@chromium.org)

2015-08-28 Thread commit-...@chromium.org via codereview.chromium.org
Patchset 1 (id:??) landed as https://crrev.com/3d7a34b5c744065a4e786645fe834ce30725c5fb Cr-Commit-Position: refs/heads/master@{#30443} https://codereview.chromium.org/1320503004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this messag

[v8-dev] Re: [heap] Move IdentityMap data structure out of heap. (issue 1320503004 by mstarzin...@chromium.org)

2015-08-28 Thread commit-...@chromium.org via codereview.chromium.org
Committed patchset #1 (id:1) https://codereview.chromium.org/1320503004/ -- -- 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 unsubscribe from this group and stop

[v8-dev] Re: [heap] Move IdentityMap data structure out of heap. (issue 1320503004 by mstarzin...@chromium.org)

2015-08-28 Thread commit-...@chromium.org via codereview.chromium.org
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1320503004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1320503004/1 https://codereview.chromium.org/1320503004/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://gro

[v8-dev] Re: [heap] Move IdentityMap data structure out of heap. (issue 1320503004 by mstarzin...@chromium.org)

2015-08-28 Thread titzer
On 2015/08/28 11:54:47, Michael Starzinger wrote: Ben: PTAL. Ross: FYI. LGTM https://codereview.chromium.org/1320503004/ -- -- 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-d