https://codereview.chromium.org/494153002/diff/1/src/ic.cc
File src/ic.cc (right):
https://codereview.chromium.org/494153002/diff/1/src/ic.cc#newcode1288
src/ic.cc:1288: }
You already found a property, so you need to make a decision. I presume
you want to return true?
So basically:
return it->property_encoding() == LookupIterator::DESCRIPTOR?
https://codereview.chromium.org/494153002/diff/1/src/ic.cc#newcode1348
src/ic.cc:1348: // LookupResult lookup(isolate());
Remove leftover
https://codereview.chromium.org/494153002/diff/1/src/ic.cc#newcode1351
src/ic.cc:1351: if (!can_store && strict_mode() == STRICT &&
object->IsGlobalObject() &&
This shouldn't depend on "can_store". Before the reference error you
could DCHECK(!can_store) though.
https://codereview.chromium.org/494153002/diff/1/src/ic.cc#newcode1352
src/ic.cc:1352: !(lookup.state() == LookupIterator::PROPERTY &&
lookup.has_property() &&
You shouldn't need to expose has_property(). Any iteration that doesn't
has_property after visiting PROPERTY should have called Next();
continuing to NOT_FOUND.
https://codereview.chromium.org/494153002/diff/1/src/ic.cc#newcode1426
src/ic.cc:1426: DCHECK(lookup->state() != LookupIterator::JSPROXY);
DCHECK_NE
https://codereview.chromium.org/494153002/diff/1/src/lookup.h
File src/lookup.h (right):
https://codereview.chromium.org/494153002/diff/1/src/lookup.h#newcode118
src/lookup.h:118: Handle<JSObject> GetPropertyTarget() const;
GetStoreTarget() ?
https://codereview.chromium.org/494153002/diff/1/src/lookup.h#newcode142
src/lookup.h:142: bool has_property() const { return has_property_; }
This shouldn't be necessary
https://codereview.chromium.org/494153002/
--
--
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.
