Changeset 49323 in webkit for trunk/JavaScriptCore/runtime/PropertyNameArray.h

Timestamp:

Oct 8, 2009, 3:02:53 PM (16 years ago)

Author:

[email protected]

Message:

Take one branch instead of two to test for JSValue().

Patch by Geoffrey Garen <​[email protected]> on 2009-10-01
Reviewed by Sam Weinig.

1.1% SunSpider speedup.

• jit/JITCall.cpp:

(JSC::JIT::compileOpCall):

• jit/JITOpcodes.cpp:

(JSC::JIT::emit_op_to_jsnumber):
(JSC::JIT::emit_op_create_arguments):

• jit/JITPropertyAccess.cpp:

(JSC::JIT::emitSlow_op_get_by_val):
(JSC::JIT::emit_op_put_by_val): Test for the empty value tag, instead
of testing for the cell tag with a 0 payload.

• runtime/JSValue.cpp:

(JSC::JSValue::description): Added support for dumping the new empty value,
and deleted values, in debug builds.

• runtime/JSValue.h:

(JSC::JSValue::JSValue()): Construct JSValue() with the empty value tag.

(JSC::JSValue::JSValue(JSCell*)): Convert null pointer to the empty value
tag, to avoid having two different c++ versions of null / empty.

(JSC::JSValue::operator bool): Test for the empty value tag, instead
of testing for the cell tag with a 0 payload.

File:

• trunk/JavaScriptCore/runtime/PropertyNameArray.h (modified) (1 diff)

trunk/JavaScriptCore/runtime/PropertyNameArray.h

@@ -24 +24 @@
 #include "CallFrame.h"
 #include "Identifier.h"
-#include "Structure.h"
+#include "StructureChain.h"
 #include <wtf/HashSet.h>
 #include <wtf/Vector.h>
 
 namespace JSC {
+    
+    class Structure;
 
     class PropertyNameArrayData : public RefCounted<PropertyNameArrayData> {