Changeset 80738 in webkit for trunk/Source/JavaScriptCore/jit/JIT.h

Timestamp:

Mar 10, 2011, 11:27:49 AM (14 years ago)

Author:

[email protected]

Message:

2011-03-10 Oliver Hunt <​[email protected]>

Reviewed by Gavin Barraclough.

jquery/manipulation.html fails after r80598
​https://bugs.webkit.org/show_bug.cgi?id=56019

Removed jQuery tests from skiplists

• platform/gtk/Skipped:
• platform/qt/Skipped:

2011-03-09 Oliver Hunt <​[email protected]>

Reviewed by Gavin Barraclough.

jquery/manipulation.html fails after r80598
​https://bugs.webkit.org/show_bug.cgi?id=56019

When linking a call, codeblock now takes ownership of the linked function
This removes the need for unlinking, and thus the incorrectness that was
showing up in these tests.

• bytecode/CodeBlock.cpp: (JSC::CodeBlock::~CodeBlock): (JSC::CodeBlock::markAggregate):
• bytecode/CodeBlock.h: (JSC::CallLinkInfo::CallLinkInfo): (JSC::CallLinkInfo::setUnlinked): (JSC::CodeBlock::addCaller):
• jit/JIT.cpp: (JSC::JIT::privateCompile): (JSC::JIT::linkCall): (JSC::JIT::linkConstruct):
• jit/JIT.h:
• runtime/Executable.cpp:
• runtime/Executable.h:

File:

• trunk/Source/JavaScriptCore/jit/JIT.h (modified) (1 diff)

trunk/Source/JavaScriptCore/jit/JIT.h

@@ -246 +246 @@
         static void linkCall(JSFunction* callee, CodeBlock* callerCodeBlock, CodeBlock* calleeCodeBlock, CodePtr, CallLinkInfo*, int callerArgCount, JSGlobalData*);
         static void linkConstruct(JSFunction* callee, CodeBlock* callerCodeBlock, CodeBlock* calleeCodeBlock, CodePtr, CallLinkInfo*, int callerArgCount, JSGlobalData*);
-        static void unlinkCallOrConstruct(CallLinkInfo*);
 
     private: