Addressed comments, thanks for the review and hints!
https://codereview.chromium.org/1221713003/diff/20001/src/elements.cc
File src/elements.cc (right):
https://codereview.chromium.org/1221713003/diff/20001/src/elements.cc#newcode1735
src/elements.cc:1735: FastHoleyObjectElementsAccessor::DeleteCommon(obj,
key, language_mode);
On 2015/07/02 13:28:42, Igor Sheludko wrote:
You can put the implementation of this to the parent if you use
ArgumentsAccessor::DeleteCommon(obj, key, language_mode);
Done.
https://codereview.chromium.org/1221713003/diff/20001/src/elements.h
File src/elements.h (right):
https://codereview.chromium.org/1221713003/diff/20001/src/elements.h#newcode24
src/elements.h:24: class SloppyArgumentsElementsAccessor;
On 2015/07/02 13:28:42, Igor Sheludko wrote:
These declarations should not be here.
Done.
https://codereview.chromium.org/1221713003/diff/20001/src/elements.h#newcode168
src/elements.h:168: ElementsKindTraits<SLOW_SLOPPY_ARGUMENTS_ELEMENTS>
;
On 2015/07/02 13:28:42, Igor Sheludko wrote:
These friend declarations should not be here. You can move them to
ElementsAccessorBase<>.
Done.
https://codereview.chromium.org/1221713003/diff/20001/src/objects.cc
File src/objects.cc (right):
https://codereview.chromium.org/1221713003/diff/20001/src/objects.cc#newcode3721
src/objects.cc:3721: allow_store_transition &=
On 2015/07/02 13:28:42, Igor Sheludko wrote:
allow_store_transition = allow_store_transition &&
Done.
https://codereview.chromium.org/1221713003/diff/20001/src/objects.h
File src/objects.h (right):
https://codereview.chromium.org/1221713003/diff/20001/src/objects.h#newcode5705
src/objects.h:5705: return elements_kind() ==
FAST_SLOPPY_ARGUMENTS_ELEMENTS ||
On 2015/07/02 13:28:42, Igor Sheludko wrote:
return IsSloppyArgumentsElements(elements_kind());
Done.
https://codereview.chromium.org/1221713003/
--
--
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.
