Review comments addressed, PTAL, thanks.
https://codereview.chromium.org/1321663003/diff/40001/src/DEPS
File src/DEPS (right):
https://codereview.chromium.org/1321663003/diff/40001/src/DEPS#newcode8
src/DEPS:8: "+src/heap/identity-map.h",
Done (thanks for doing the move!)
https://codereview.chromium.org/1321663003/diff/40001/src/heap/identity-map.h
File src/heap/identity-map.h (right):
https://codereview.chromium.org/1321663003/diff/40001/src/heap/identity-map.h#newcode9
src/heap/identity-map.h:9: // Do not include anything from src/heap
here!
On 2015/08/28 12:21:04, Michael Starzinger wrote:
Likewise.
Done.
https://codereview.chromium.org/1321663003/diff/40001/src/interpreter/bytecode-array-builder.cc
File src/interpreter/bytecode-array-builder.cc (right):
https://codereview.chromium.org/1321663003/diff/40001/src/interpreter/bytecode-array-builder.cc#newcode55
src/interpreter/bytecode-array-builder.cc:55: if (constants_.size() ==
0) {
On 2015/08/28 12:21:04, Michael Starzinger wrote:
This special-casing is already part of Factory::NewFixedArray, it
should fall
out naturally of the code below.
Perfect! thanks, done.
https://codereview.chromium.org/1321663003/diff/40001/src/interpreter/bytecode-array-builder.h
File src/interpreter/bytecode-array-builder.h (right):
https://codereview.chromium.org/1321663003/diff/40001/src/interpreter/bytecode-array-builder.h#newcode27
src/interpreter/bytecode-array-builder.h:27: explicit
BytecodeArrayBuilder(Isolate* isolate, Zone* zone);
On 2015/08/28 12:21:04, Michael Starzinger wrote:
nit: No longer needs to be marked as explicit.
Done.
https://codereview.chromium.org/1321663003/diff/40001/src/interpreter/bytecode-array-builder.h#newcode79
src/interpreter/bytecode-array-builder.h:79: std::vector<uint8_t>
bytecodes_;
On 2015/08/28 12:21:04, Michael Starzinger wrote:
Same comment from below applies here.
Done.
https://codereview.chromium.org/1321663003/diff/40001/src/interpreter/bytecode-array-builder.h#newcode83
src/interpreter/bytecode-array-builder.h:83: std::vector<Handle<Object>>
constants_;
On 2015/08/28 12:21:04, Michael Starzinger wrote:
Could we use a ZoneVector here, that would provide us with accounting
for the
(zone) memory footprint of the interpreter later (i.e. similar to
--turbo-stats)
Done.
https://codereview.chromium.org/1321663003/
--
--
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 emails from it, send an email
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
