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

Timestamp:

Aug 18, 2002, 12:21:26 AM (23 years ago)

Author:

mjs

Message:

Separated Completion from Value and made it a pure stack
object. This removed another 160,000 of the remaining 580,000
garbage collected object allocations.

6% speed increas on cvs-js-performance test.

• kjs/completion.cpp: Added. New implementation that doesn't require a ValueImp *. (Completion::Completion): (Completion::complType): (Completion::value): (Completion::target): (Completion::isValueCompletion):
• kjs/completion.h: Added.
• kjs/function.cpp: (GlobalFuncImp::call): Removed some (apparently mistaken) uses of Completion as a Value.
• kjs/internal.cpp:
• kjs/internal.h:
• kjs/types.cpp: Removed Completion stuff.
• kjs/types.h: Removed Completion stuff.
• JavaScriptCore.pbproj/project.pbxproj: Added new header.

File:

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

trunk/JavaScriptCore/kjs/function.cpp

@@ -417 +417 @@
           delete progNode;
       if (c.complType() == ReturnValue)
-          return c;
+          return c.value();
       // ### setException() on throw?
       else if (c.complType() == Normal) {
@@ -425 +425 @@
               return Undefined();
       } else {
-          return c;
+          return Undefined();
       }
     }