Revision: 17462
Author: [email protected]
Date: Mon Nov 4 16:39:58 2013 UTC
Log: Restore intended LoadIC behavior for JSBuiltinsObject
[email protected]
Review URL: https://codereview.chromium.org/48343004
http://code.google.com/p/v8/source/detail?r=17462
Modified:
/branches/bleeding_edge/src/ic.cc
=======================================
--- /branches/bleeding_edge/src/ic.cc Mon Oct 28 16:32:56 2013 UTC
+++ /branches/bleeding_edge/src/ic.cc Mon Nov 4 16:39:58 2013 UTC
@@ -370,6 +370,18 @@
void IC::UpdateState(Handle<Object> receiver, Handle<Object> name) {
if (!name->IsString()) return;
+
+ // The builtins object is special. It only changes when JavaScript
+ // builtins are loaded lazily. It is important to keep inline
+ // caches for the builtins object monomorphic. Therefore, if we get
+ // an inline cache miss for the builtins object after lazily loading
+ // JavaScript builtins, we return uninitialized as the state to
+ // force the inline cache back to monomorphic state.
+ if (receiver->IsJSBuiltinsObject()) {
+ state_ = UNINITIALIZED;
+ return;
+ }
+
if (state() != MONOMORPHIC) {
if (state() == POLYMORPHIC && receiver->IsHeapObject()) {
TryRemoveInvalidHandlers(
@@ -387,14 +399,6 @@
receiver, Handle<String>::cast(name))) {
return MarkMonomorphicPrototypeFailure();
}
-
- // The builtins object is special. It only changes when JavaScript
- // builtins are loaded lazily. It is important to keep inline
- // caches for the builtins object monomorphic. Therefore, if we get
- // an inline cache miss for the builtins object after lazily loading
- // JavaScript builtins, we return uninitialized as the state to
- // force the inline cache back to monomorphic state.
- if (receiver->IsJSBuiltinsObject()) state_ = UNINITIALIZED;
}
--
--
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/groups/opt_out.
