Changeset 27068 in webkit for trunk/JavaScriptCore/kjs/function.cpp

Timestamp:

Oct 25, 2007, 2:37:45 PM (18 years ago)

Author:

ggaren

Message:

Reviewed by Oliver Hunt.

Fixed ​http://bugs.webkit.org/show_bug.cgi?id=15694
Shrink the size of an activation object by 1 word

This is in preparation for adding a symbol table to the activation
object.

The basic strategy here is to rely on the mutual exclusion between
the arguments object pointer and the function pointer (you only need
the latter in order to create the former), and store them in the same
place. The LazyArgumentsObject class encapsulates this strategy.

Also inlined the ArgumentsImp constructor, for good measure.

SunSpider reports no regression. Regression tests pass.

• JavaScriptCore.xcodeproj/project.pbxproj:
• kjs/Context.cpp: (KJS::Context::~Context):
• kjs/function.cpp: (KJS::ActivationImp::LazyArgumentsObject::createArgumentsObject): (KJS::ActivationImp::LazyArgumentsObject::mark): (KJS::): (KJS::ActivationImp::argumentsGetter): (KJS::ActivationImp::mark):
• kjs/function.h: (KJS::ActivationImp::LazyArgumentsObject::LazyArgumentsObject): (KJS::ActivationImp::LazyArgumentsObject::getOrCreate): (KJS::ActivationImp::LazyArgumentsObject::resetArguments): (KJS::ActivationImp::LazyArgumentsObject::setArgumentsObject): (KJS::ActivationImp::LazyArgumentsObject::argumentsObject): (KJS::ActivationImp::LazyArgumentsObject::setFunction): (KJS::ActivationImp::LazyArgumentsObject::function): (KJS::ActivationImp::LazyArgumentsObject::createdArgumentsObject): (KJS::ActivationImp::LazyArgumentsObject::): (KJS::ActivationImp::ActivationImp::ActivationImp): (KJS::ActivationImp::resetArguments):

File:

• trunk/JavaScriptCore/kjs/function.cpp (modified) (2 diffs)

trunk/JavaScriptCore/kjs/function.cpp

@@ -391 +391 @@
 // ------------------------------ ActivationImp --------------------------------
 
+void ActivationImp::LazyArgumentsObject::createArgumentsObject(ExecState* exec, ActivationImp* activationImp)
+{
+    setArgumentsObject(new Arguments(exec, function(), arguments, activationImp));
+
+    // The arguments list is only needed to create the arguments object, so 
+    // discard it now. This prevents lists of Lists from building up, waiting 
+    // to be garbage collected.
+    arguments.reset();
+}
+
+void ActivationImp::LazyArgumentsObject::mark()
+{
+    JSObject* o;
+    if (createdArgumentsObject())
+        o = argumentsObject();
+    else
+        o = function();
+
+    if (!o->marked())
+        o->mark();
+}
+
 const ClassInfo ActivationImp::info = {"Activation", 0, 0, 0};
 
-// ECMA 10.1.6
-ActivationImp::ActivationImp(FunctionImp* function, const List& arguments)
-    : _function(function), _arguments(arguments), _argumentsObject(0)
-{
-  // FIXME: Do we need to support enumerating the arguments property?
-}
-
 JSValue* ActivationImp::argumentsGetter(ExecState* exec, JSObject*, const Identifier&, const PropertySlot& slot)
 {
   ActivationImp* thisObj = static_cast<ActivationImp*>(slot.slotBase());
-
-  // default: return builtin arguments array
-  if (!thisObj->_argumentsObject)
-    thisObj->createArgumentsObject(exec);
-  
-  return thisObj->_argumentsObject;
+  return thisObj->m_lazyArgumentsObject.getOrCreate(exec, thisObj);
 }
 
@@ -453 +463 @@
 void ActivationImp::mark()
 {
-    if (_function && !_function->marked()) 
-        _function->mark();
-    if (_argumentsObject && !_argumentsObject->marked())
-        _argumentsObject->mark();
+    m_lazyArgumentsObject.mark();
     JSObject::mark();
-}
-
-void ActivationImp::createArgumentsObject(ExecState* exec)
-{
-  _argumentsObject = new Arguments(exec, _function, _arguments, const_cast<ActivationImp*>(this));
-  // The arguments list is only needed to create the arguments object, so discard it now
-  _arguments.reset();
 }