https://codereview.chromium.org/138953018/diff/110001/src/spaces.cc
File src/spaces.cc (right):
https://codereview.chromium.org/138953018/diff/110001/src/spaces.cc#newcode2131
src/spaces.cc:2131: FreeListNode** n = &t;
On 2014/02/13 12:35:49, jarin wrote:
Now I am wondering why we can't use t directly here (replace '*n' with
't' in
the loop below).
For sure we can, legacy code. We could have done that before as well.
https://codereview.chromium.org/138953018/diff/110001/src/spaces.cc#newcode2314
src/spaces.cc:2314: huge_list_.set_top(top_node);
On 2014/02/13 12:35:49, jarin wrote:
This is a bit funny - which part of the loop reads huge_list_?
This store to top is not needed here since it is done outside the loop.
https://codereview.chromium.org/138953018/
--
--
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/groups/opt_out.
