From: verwaest@chromium.org Date: Tue, 22 Jan 2013 09:48:59 +0000 (+0000) Subject: Merge Store from Keyed and Named store. X-Git-Tag: upstream/4.7.83~15239 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=2c070e23007b87cc349c6ecdec70dbc2ce6a37e3;p=platform%2Fupstream%2Fv8.git Merge Store from Keyed and Named store. Review URL: https://chromiumcodereview.appspot.com/12036017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13461 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- diff --git a/src/ic.cc b/src/ic.cc index 5d6f1f1..b90436a 100644 --- a/src/ic.cc +++ b/src/ic.cc @@ -351,11 +351,9 @@ void IC::Clear(Address address) { switch (target->kind()) { case Code::LOAD_IC: return LoadIC::Clear(address, target); - case Code::KEYED_LOAD_IC: - return KeyedLoadIC::Clear(address, target); + case Code::KEYED_LOAD_IC: return KeyedLoadIC::Clear(address, target); case Code::STORE_IC: return StoreIC::Clear(address, target); - case Code::KEYED_STORE_IC: - return KeyedStoreIC::Clear(address, target); + case Code::KEYED_STORE_IC: return KeyedStoreIC::Clear(address, target); case Code::CALL_IC: return CallIC::Clear(address, target); case Code::KEYED_CALL_IC: return KeyedCallIC::Clear(address, target); case Code::COMPARE_IC: return CompareIC::Clear(address, target); @@ -387,13 +385,13 @@ void KeyedLoadIC::Clear(Address address, Code* target) { // Make sure to also clear the map used in inline fast cases. If we // do not clear these maps, cached code can keep objects alive // through the embedded maps. - SetTargetAtAddress(address, initialize_stub()); + SetTargetAtAddress(address, *initialize_stub()); } void LoadIC::Clear(Address address, Code* target) { if (target->ic_state() == UNINITIALIZED) return; - SetTargetAtAddress(address, initialize_stub()); + SetTargetAtAddress(address, *initialize_stub()); } @@ -401,8 +399,8 @@ void StoreIC::Clear(Address address, Code* target) { if (target->ic_state() == UNINITIALIZED) return; SetTargetAtAddress(address, (Code::GetStrictMode(target->extra_ic_state()) == kStrictMode) - ? initialize_stub_strict() - : initialize_stub()); + ? *initialize_stub_strict() + : *initialize_stub()); } @@ -410,8 +408,8 @@ void KeyedStoreIC::Clear(Address address, Code* target) { if (target->ic_state() == UNINITIALIZED) return; SetTargetAtAddress(address, (Code::GetStrictMode(target->extra_ic_state()) == kStrictMode) - ? initialize_stub_strict() - : initialize_stub()); + ? *initialize_stub_strict() + : *initialize_stub()); } @@ -1313,34 +1311,34 @@ static bool LookupForWrite(Handle receiver, } -MaybeObject* StoreIC::Store(State state, - StrictModeFlag strict_mode, - Handle object, - Handle name, - Handle value) { - if (!object->IsJSObject()) { - // Handle proxies. - if (object->IsJSProxy()) { - return JSProxy::cast(*object)-> - SetProperty(*name, *value, NONE, strict_mode); - } +MaybeObject* IC::Store(State state, + StrictModeFlag strict_mode, + Handle object, + Handle name, + Handle value, + JSReceiver::StoreFromKeyed store_mode) { + // Handle proxies. + if (object->IsJSProxy()) { + return JSProxy::cast(*object)-> + SetProperty(*name, *value, NONE, strict_mode); + } - // If the object is undefined or null it's illegal to try to set any - // properties on it; throw a TypeError in that case. - if (object->IsUndefined() || object->IsNull()) { - return TypeError("non_object_property_store", object, name); - } + // If the object is undefined or null it's illegal to try to set any + // properties on it; throw a TypeError in that case. + if (object->IsUndefined() || object->IsNull()) { + return TypeError("non_object_property_store", object, name); + } - // The length property of string values is read-only. Throw in strict mode. - if (strict_mode == kStrictMode && object->IsString() && - name->Equals(isolate()->heap()->length_symbol())) { - return TypeError("strict_read_only_property", object, name); - } - // Ignore other stores where the receiver is not a JSObject. - // TODO(1475): Must check prototype chains of object wrappers. - return *value; + // The length property of string values is read-only. Throw in strict mode. + if (strict_mode == kStrictMode && object->IsString() && + name->Equals(isolate()->heap()->length_symbol())) { + return TypeError("strict_read_only_property", object, name); } + // Ignore other stores where the receiver is not a JSObject. + // TODO(1475): Must check prototype chains of object wrappers. + if (!object->IsJSObject()) return *value; + Handle receiver = Handle::cast(object); // Check if the given name is an array index. @@ -1354,75 +1352,63 @@ MaybeObject* StoreIC::Store(State state, // Observed objects are always modified through the runtime. if (FLAG_harmony_observation && receiver->map()->is_observed()) { - return receiver->SetProperty(*name, *value, NONE, strict_mode); + return receiver->SetProperty(*name, *value, NONE, strict_mode, store_mode); } // Use specialized code for setting the length of arrays with fast - // properties. Slow properties might indicate redefinition of the - // length property. - if (receiver->IsJSArray() && + // properties. Slow properties might indicate redefinition of the length + // property. + if (FLAG_use_ic && + receiver->IsJSArray() && name->Equals(isolate()->heap()->length_symbol()) && Handle::cast(receiver)->AllowsSetElementsLength() && - receiver->HasFastProperties()) { -#ifdef DEBUG - if (FLAG_trace_ic) PrintF("[StoreIC : +#length /array]\n"); -#endif + receiver->HasFastProperties() && + kind() != Code::KEYED_STORE_IC) { Handle stub = (strict_mode == kStrictMode) ? isolate()->builtins()->StoreIC_ArrayLength_Strict() : isolate()->builtins()->StoreIC_ArrayLength(); set_target(*stub); - return receiver->SetProperty(*name, *value, NONE, strict_mode); + TRACE_IC("StoreIC", name, state, *stub); + return receiver->SetProperty(*name, *value, NONE, strict_mode, store_mode); } - // Lookup the property locally in the receiver. - if (!receiver->IsJSGlobalProxy()) { - LookupResult lookup(isolate()); - - if (LookupForWrite(receiver, name, &lookup)) { - if (FLAG_use_ic) { // Generate a stub for this store. - UpdateCaches(&lookup, state, strict_mode, receiver, name, value); - } - } else { - // Strict mode doesn't allow setting non-existent global property - // or an assignment to a read only property. - if (strict_mode == kStrictMode) { - if (lookup.IsProperty() && lookup.IsReadOnly()) { - return TypeError("strict_read_only_property", object, name); - } else if (IsContextual(object)) { - return ReferenceError("not_defined", name); - } - } + if (receiver->IsJSGlobalProxy()) { + if (FLAG_use_ic && kind() != Code::KEYED_STORE_IC) { + // Generate a generic stub that goes to the runtime when we see a global + // proxy as receiver. + Handle stub = (strict_mode == kStrictMode) + ? global_proxy_stub_strict() + : global_proxy_stub(); + set_target(*stub); + TRACE_IC("StoreIC", name, state, *stub); } + return receiver->SetProperty(*name, *value, NONE, strict_mode, store_mode); } - if (receiver->IsJSGlobalProxy()) { - // TODO(ulan): find out why we patch this site even with --no-use-ic - // Generate a generic stub that goes to the runtime when we see a global - // proxy as receiver. - Handle stub = (strict_mode == kStrictMode) - ? global_proxy_stub_strict() - : global_proxy_stub(); - if (target() != *stub) { - set_target(*stub); - TRACE_IC("StoreIC", name, state, target()); + LookupResult lookup(isolate()); + if (LookupForWrite(receiver, name, &lookup)) { + if (FLAG_use_ic) { + UpdateStoreCaches(&lookup, state, strict_mode, receiver, name, value); } + } else if (strict_mode == kStrictMode && + !lookup.IsFound() && + IsContextual(object)) { + // Strict mode doesn't allow setting non-existent global property + // or an assignment to a read only property. + return ReferenceError("not_defined", name); } // Set the property. - return receiver->SetProperty(*name, - *value, - NONE, - strict_mode, - JSReceiver::CERTAINLY_NOT_STORE_FROM_KEYED); + return receiver->SetProperty(*name, *value, NONE, strict_mode, store_mode); } -void StoreIC::UpdateCaches(LookupResult* lookup, - State state, - StrictModeFlag strict_mode, - Handle receiver, - Handle name, - Handle value) { +void StoreIC::UpdateStoreCaches(LookupResult* lookup, + State state, + StrictModeFlag strict_mode, + Handle receiver, + Handle name, + Handle value) { ASSERT(!receiver->IsJSGlobalProxy()); ASSERT(StoreICableLookup(lookup)); ASSERT(lookup->IsFound()); @@ -1531,7 +1517,7 @@ void StoreIC::UpdateCaches(LookupResult* lookup, } isolate()->stub_cache()->Set(*name, receiver->map(), *code); set_target((strict_mode == kStrictMode) - ? megamorphic_stub_strict() + ? *megamorphic_stub_strict() : *megamorphic_stub()); } break; @@ -1891,43 +1877,12 @@ MaybeObject* KeyedStoreIC::Store(State state, if (key->IsSymbol()) { Handle name = Handle::cast(key); - - // Handle proxies. - if (object->IsJSProxy()) { - return JSProxy::cast(*object)->SetProperty( - *name, *value, NONE, strict_mode); - } - - // If the object is undefined or null it's illegal to try to set any - // properties on it; throw a TypeError in that case. - if (object->IsUndefined() || object->IsNull()) { - return TypeError("non_object_property_store", object, name); - } - - // Ignore stores where the receiver is not a JSObject. - if (!object->IsJSObject()) return *value; - Handle receiver = Handle::cast(object); - - // Check if the given name is an array index. - uint32_t index; - if (name->AsArrayIndex(&index)) { - Handle result = - JSObject::SetElement(receiver, index, value, NONE, strict_mode); - RETURN_IF_EMPTY_HANDLE(isolate(), result); - return *value; - } - - // Update inline cache and stub cache. - if (FLAG_use_ic && !receiver->IsJSGlobalProxy() && - !(FLAG_harmony_observation && receiver->map()->is_observed())) { - LookupResult lookup(isolate()); - if (LookupForWrite(receiver, name, &lookup)) { - UpdateCaches(&lookup, state, strict_mode, receiver, name, value); - } - } - - // Set the property. - return receiver->SetProperty(*name, *value, NONE, strict_mode); + return IC::Store(state, + strict_mode, + object, + name, + value, + JSReceiver::MAY_BE_STORE_FROM_KEYED); } // Do not use ICs for objects that require access checks (including @@ -1966,12 +1921,12 @@ MaybeObject* KeyedStoreIC::Store(State state, } -void KeyedStoreIC::UpdateCaches(LookupResult* lookup, - State state, - StrictModeFlag strict_mode, - Handle receiver, - Handle name, - Handle value) { +void KeyedStoreIC::UpdateStoreCaches(LookupResult* lookup, + State state, + StrictModeFlag strict_mode, + Handle receiver, + Handle name, + Handle value) { ASSERT(!receiver->IsJSGlobalProxy()); ASSERT(StoreICableLookup(lookup)); ASSERT(lookup->IsFound()); diff --git a/src/ic.h b/src/ic.h index c95f2e3..784512a 100644 --- a/src/ic.h +++ b/src/ic.h @@ -136,6 +136,15 @@ class IC { Handle object, Handle name); + MUST_USE_RESULT MaybeObject* Store( + State state, + StrictModeFlag strict_mode, + Handle object, + Handle name, + Handle value, + JSReceiver::StoreFromKeyed store_mode = + JSReceiver::CERTAINLY_NOT_STORE_FROM_KEYED); + protected: virtual Handle pre_monomorphic_stub() { UNREACHABLE(); @@ -145,6 +154,10 @@ class IC { UNREACHABLE(); return Handle::null(); } + virtual Handle megamorphic_stub_strict() { + UNREACHABLE(); + return Handle::null(); + } virtual Handle generic_stub() const { UNREACHABLE(); return Handle::null(); @@ -153,12 +166,29 @@ class IC { UNREACHABLE(); return Code::STUB; } + virtual Handle global_proxy_stub() { + UNREACHABLE(); + return Handle::null(); + } + virtual Handle global_proxy_stub_strict() { + UNREACHABLE(); + return Handle::null(); + } + virtual void UpdateLoadCaches(LookupResult* lookup, State state, Handle object, Handle name) { UNREACHABLE(); } + virtual void UpdateStoreCaches(LookupResult* lookup, + State state, + StrictModeFlag strict_mode, + Handle receiver, + Handle name, + Handle value) { + UNREACHABLE(); + } Address fp() const { return fp_; } Address pc() const { return *pc_address_; } Isolate* isolate() const { return isolate_; } @@ -378,9 +408,8 @@ class LoadIC: public IC { private: // Stub accessors. - static Code* initialize_stub() { - return Isolate::Current()->builtins()->builtin( - Builtins::kLoadIC_Initialize); + static Handle initialize_stub() { + return Isolate::Current()->builtins()->LoadIC_Initialize(); } virtual Handle pre_monomorphic_stub() { return isolate()->builtins()->LoadIC_PreMonomorphic(); @@ -565,9 +594,8 @@ class KeyedLoadIC: public KeyedIC { private: // Stub accessors. - static Code* initialize_stub() { - return Isolate::Current()->builtins()->builtin( - Builtins::kKeyedLoadIC_Initialize); + static Handle initialize_stub() { + return Isolate::Current()->builtins()->KeyedLoadIC_Initialize(); } virtual Handle pre_monomorphic_stub() { return isolate()->builtins()->KeyedLoadIC_PreMonomorphic(); @@ -591,12 +619,6 @@ class StoreIC: public IC { ASSERT(target()->is_store_stub()); } - MUST_USE_RESULT MaybeObject* Store(State state, - StrictModeFlag strict_mode, - Handle object, - Handle name, - Handle value); - // Code generators for stub routines. Only called once at startup. static void GenerateInitialize(MacroAssembler* masm) { GenerateMiss(masm); } static void GenerateMiss(MacroAssembler* masm); @@ -612,17 +634,28 @@ class StoreIC: public IC { virtual Handle megamorphic_stub() { return isolate()->builtins()->StoreIC_Megamorphic(); } + // Stub accessors. + virtual Handle megamorphic_stub_strict() { + return isolate()->builtins()->StoreIC_Megamorphic_Strict(); + } + virtual Handle global_proxy_stub() { + return isolate()->builtins()->StoreIC_GlobalProxy(); + } + virtual Handle global_proxy_stub_strict() { + return isolate()->builtins()->StoreIC_GlobalProxy_Strict(); + } + - private: // Update the inline cache and the global stub cache based on the // lookup result. - void UpdateCaches(LookupResult* lookup, - State state, - StrictModeFlag strict_mode, - Handle receiver, - Handle name, - Handle value); + virtual void UpdateStoreCaches(LookupResult* lookup, + State state, + StrictModeFlag strict_mode, + Handle receiver, + Handle name, + Handle value); + private: void set_target(Code* code) { // Strict mode must be preserved across IC patching. ASSERT(Code::GetStrictMode(code->extra_ic_state()) == @@ -630,26 +663,12 @@ class StoreIC: public IC { IC::set_target(code); } - // Stub accessors. - Code* megamorphic_stub_strict() { - return isolate()->builtins()->builtin( - Builtins::kStoreIC_Megamorphic_Strict); - } - static Code* initialize_stub() { - return Isolate::Current()->builtins()->builtin( - Builtins::kStoreIC_Initialize); + static Handle initialize_stub() { + return Isolate::Current()->builtins()->StoreIC_Initialize(); } - static Code* initialize_stub_strict() { - return Isolate::Current()->builtins()->builtin( - Builtins::kStoreIC_Initialize_Strict); + static Handle initialize_stub_strict() { + return Isolate::Current()->builtins()->StoreIC_Initialize_Strict(); } - Handle global_proxy_stub() { - return isolate()->builtins()->StoreIC_GlobalProxy(); - } - Handle global_proxy_stub_strict() { - return isolate()->builtins()->StoreIC_GlobalProxy_Strict(); - } - static void Clear(Address address, Code* target); friend class IC; @@ -706,15 +725,22 @@ class KeyedStoreIC: public KeyedIC { StrictModeFlag strict_mode, KeyedAccessGrowMode grow_mode); - private: // Update the inline cache. - void UpdateCaches(LookupResult* lookup, - State state, - StrictModeFlag strict_mode, - Handle receiver, - Handle name, - Handle value); + virtual void UpdateStoreCaches(LookupResult* lookup, + State state, + StrictModeFlag strict_mode, + Handle receiver, + Handle name, + Handle value); + + virtual Handle megamorphic_stub() { + return isolate()->builtins()->KeyedStoreIC_Generic(); + } + virtual Handle megamorphic_stub_strict() { + return isolate()->builtins()->KeyedStoreIC_Generic_Strict(); + } + private: void set_target(Code* code) { // Strict mode must be preserved across IC patching. ASSERT(Code::GetStrictMode(code->extra_ic_state()) == @@ -723,19 +749,11 @@ class KeyedStoreIC: public KeyedIC { } // Stub accessors. - static Code* initialize_stub() { - return Isolate::Current()->builtins()->builtin( - Builtins::kKeyedStoreIC_Initialize); - } - static Code* initialize_stub_strict() { - return Isolate::Current()->builtins()->builtin( - Builtins::kKeyedStoreIC_Initialize_Strict); + static Handle initialize_stub() { + return Isolate::Current()->builtins()->KeyedStoreIC_Initialize(); } - Handle megamorphic_stub() { - return isolate()->builtins()->KeyedStoreIC_Generic(); - } - Handle megamorphic_stub_strict() { - return isolate()->builtins()->KeyedStoreIC_Generic_Strict(); + static Handle initialize_stub_strict() { + return Isolate::Current()->builtins()->KeyedStoreIC_Initialize_Strict(); } Handle generic_stub() const { return isolate()->builtins()->KeyedStoreIC_Generic();