Changeset 195876 in webkit for trunk/Source/JavaScriptCore/runtime/Executable.cpp

Timestamp:

Jan 29, 2016, 6:05:17 PM (10 years ago)

Author:

[email protected]

Message:

Pack FunctionExecutable and UnlinkedFunctionExecutable harder
​https://bugs.webkit.org/show_bug.cgi?id=153687

Reviewed by Andreas Kling.

This patch reduces FunctionExecutable from 120 to 104 bytes.
This patch reduces UnlinkedFunctionExecutable from 144 to 136 bytes.

• bytecode/ExecutableInfo.h:
• bytecode/UnlinkedFunctionExecutable.cpp:

(JSC::UnlinkedFunctionExecutable::UnlinkedFunctionExecutable):

• bytecode/UnlinkedFunctionExecutable.h:
• parser/ParserModes.h:

(JSC::functionNameScopeIsDynamic):

• runtime/Executable.cpp:

(JSC::ScriptExecutable::ScriptExecutable):

• runtime/Executable.h:

(JSC::ScriptExecutable::needsActivation):
(JSC::ScriptExecutable::isArrowFunctionContext):
(JSC::ScriptExecutable::isStrictMode):
(JSC::ScriptExecutable::derivedContextType):
(JSC::ScriptExecutable::ecmaMode):
(JSC::ScriptExecutable::finishCreation):

File:

• trunk/Source/JavaScriptCore/runtime/Executable.cpp (modified) (2 diffs)

trunk/Source/JavaScriptCore/runtime/Executable.cpp

@@ -134 +134 @@
 ScriptExecutable::ScriptExecutable(Structure* structure, VM& vm, const SourceCode& source, bool isInStrictContext, DerivedContextType derivedContextType, bool isInArrowFunctionContext)
     : ExecutableBase(vm, structure, NUM_PARAMETERS_NOT_COMPILED)
-    , m_source(source)
     , m_features(isInStrictContext ? StrictModeFeature : 0)
+    , m_didTryToEnterInLoop(false)
     , m_hasCapturedVariables(false)
     , m_neverInline(false)
-    , m_didTryToEnterInLoop(false)
-    , m_derivedContextType(derivedContextType)
+    , m_neverOptimize(false)
     , m_isArrowFunctionContext(isInArrowFunctionContext)
+    , m_derivedContextType(static_cast<unsigned>(derivedContextType))
     , m_overrideLineNumber(-1)
     , m_firstLine(-1)
@@ -148 +148 @@
     , m_typeProfilingStartOffset(UINT_MAX)
     , m_typeProfilingEndOffset(UINT_MAX)
+    , m_source(source)
 {
 }