Changeset 172961 in webkit for trunk/Source/JavaScriptCore/dfg/DFGLazyJSValue.cpp

Timestamp:

Aug 26, 2014, 9:46:10 AM (11 years ago)

Author:

[email protected]

Message:

Unreviewed, rolling out r172940.
​https://bugs.webkit.org/show_bug.cgi?id=136256

Caused assertions on fast/storage/serialized-script-
value.html, and possibly flakiness on more tests (Requested by
ap on #webkit).

Reverted changeset:

"FTL should be able to do polymorphic call inlining"
​https://bugs.webkit.org/show_bug.cgi?id=135145
​http://trac.webkit.org/changeset/172940

File:

• trunk/Source/JavaScriptCore/dfg/DFGLazyJSValue.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/dfg/DFGLazyJSValue.cpp

@@ -114 +114 @@
 }
 
-uintptr_t LazyJSValue::switchLookupValue(SwitchKind kind) const
-{
-    // NB. Not every kind of JSValue will be able to give you a switch lookup
-    // value, and this method will assert, or do bad things, if you use it
-    // for a kind of value that can't.
-    switch (m_kind) {
-    case KnownValue:
-        switch (kind) {
-        case SwitchImm:
-            return value()->value().asInt32();
-        case SwitchCell:
-            return bitwise_cast<uintptr_t>(value()->value().asCell());
-        default:
-            RELEASE_ASSERT_NOT_REACHED();
-            return 0;
-        }
-    case SingleCharacterString:
-        switch (kind) {
-        case SwitchChar:
-            return character();
-        default:
-            RELEASE_ASSERT_NOT_REACHED();
-            return 0;
-        }
-    default:
-        RELEASE_ASSERT_NOT_REACHED();
-        return 0;
-    }
-}
-
 void LazyJSValue::dumpInContext(PrintStream& out, DumpContext* context) const
 {