Crankshaft stub?
https://codereview.chromium.org/279423005/diff/1/src/ast.h
File src/ast.h (right):
https://codereview.chromium.org/279423005/diff/1/src/ast.h#newcode1733
src/ast.h:1733: class AllocationSiteInfo: public ZoneObject {
CallInfo? Don't wrap this puppy until we actually have another usage
where you need this abstraction.
https://codereview.chromium.org/279423005/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):
https://codereview.chromium.org/279423005/diff/1/src/hydrogen.cc#newcode8502
src/hydrogen.cc:8502: if (argument_count == 0) {
ASSERT(argument_count == 0); and remove the if.
https://codereview.chromium.org/279423005/diff/1/src/hydrogen.h
File src/hydrogen.h (right):
https://codereview.chromium.org/279423005/diff/1/src/hydrogen.h#newcode1296
src/hydrogen.h:1296: static const int kLoopUnfoldLimit = 8;
I've always been suspicious of this name. You should call this
kElementLoopUnrollThreshold.
https://codereview.chromium.org/279423005/diff/1/src/ia32/code-stubs-ia32.cc
File src/ia32/code-stubs-ia32.cc (right):
https://codereview.chromium.org/279423005/diff/1/src/ia32/code-stubs-ia32.cc#newcode2550
src/ia32/code-stubs-ia32.cc:2550: __ int3();
I think there might be a macro for this.
https://codereview.chromium.org/279423005/diff/1/src/ic.cc
File src/ic.cc (right):
https://codereview.chromium.org/279423005/diff/1/src/ic.cc#newcode513
src/ic.cc:513: // Install default stub with the immutable parts of
existing state.
Code below is unreachable. Just make above
ASSERT(existing_state.stub_type() == MONOMORPHIC_ARRAY);
return;
https://codereview.chromium.org/279423005/diff/1/src/ic.h
File src/ic.h (right):
https://codereview.chromium.org/279423005/diff/1/src/ic.h#newcode389
src/ic.h:389: CallType call_type,
fits on a single line?
https://codereview.chromium.org/279423005/diff/1/src/objects-visiting-inl.h
File src/objects-visiting-inl.h (right):
https://codereview.chromium.org/279423005/diff/1/src/objects-visiting-inl.h#newcode298
src/objects-visiting-inl.h:298: target->kind() == Code::CALL_IC) ||
This code can be removed.
https://codereview.chromium.org/279423005/diff/1/src/type-info.cc
File src/type-info.cc (right):
https://codereview.chromium.org/279423005/diff/1/src/type-info.cc#newcode137
src/type-info.cc:137: return
Handle<JSFunction>(isolate()->native_context()->array_function());
From our conversation: Don't clear the function the TypeVector for
call_new if it is the array function, since that won't leak.
https://codereview.chromium.org/279423005/
--
--
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/d/optout.
