[v8-dev] Re: Use OrderedHashTables as the backing store of JSSet and JSMap (issue 225183009)

2014-04-08 Thread adamk
Committed patchset #2 manually as r20585 (presubmit successful). https://codereview.chromium.org/225183009/ -- -- 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 u

[v8-dev] Re: Use OrderedHashTables as the backing store of JSSet and JSMap (issue 225183009)

2014-04-07 Thread adamk
https://codereview.chromium.org/225183009/diff/1/src/objects.h File src/objects.h (right): https://codereview.chromium.org/225183009/diff/1/src/objects.h#newcode1080 src/objects.h:1080: V(OrderedHashTable) \ On 2014/04/07 12:37:31, Michael Starzinger wrote: Since the Or

[v8-dev] Re: Use OrderedHashTables as the backing store of JSSet and JSMap (issue 225183009)

2014-04-07 Thread mstarzinger
LGTM. Just one comment. https://codereview.chromium.org/225183009/diff/1/src/objects.h File src/objects.h (right): https://codereview.chromium.org/225183009/diff/1/src/objects.h#newcode1080 src/objects.h:1080: V(OrderedHashTable) \ Since the OrderedHashTable is just the