Changeset 180257 in webkit for trunk/Source/JavaScriptCore/dfg/DFGOSRExit.h

Timestamp:

Feb 17, 2015, 11:04:10 PM (11 years ago)

Author:

[email protected]

Message:

Clean up OSRExit's considerAddingAsFrequentExitSite()
​https://bugs.webkit.org/show_bug.cgi?id=141690

Patch by Benjamin Poulain <​[email protected]> on 2015-02-17
Reviewed by Anders Carlsson.

Looks like some code was removed from CodeBlock::tallyFrequentExitSites()
and the OSRExit were left untouched.

This patch cleans up the two loops and remove the boolean return
on considerAddingAsFrequentExitSite().

• bytecode/CodeBlock.cpp:

(JSC::CodeBlock::tallyFrequentExitSites):

• dfg/DFGOSRExit.h:

(JSC::DFG::OSRExit::considerAddingAsFrequentExitSite):

• dfg/DFGOSRExitBase.cpp:

(JSC::DFG::OSRExitBase::considerAddingAsFrequentExitSiteSlow):

• dfg/DFGOSRExitBase.h:

(JSC::DFG::OSRExitBase::considerAddingAsFrequentExitSite):

• ftl/FTLOSRExit.h:

(JSC::FTL::OSRExit::considerAddingAsFrequentExitSite):

File:

• trunk/Source/JavaScriptCore/dfg/DFGOSRExit.h (modified) (1 diff)

trunk/Source/JavaScriptCore/dfg/DFGOSRExit.h

@@ -105 +105 @@
     RefPtr<ValueRecoveryOverride> m_valueRecoveryOverride;
     
-    bool considerAddingAsFrequentExitSite(CodeBlock* profiledCodeBlock)
+    void considerAddingAsFrequentExitSite(CodeBlock* profiledCodeBlock)
     {
-        return OSRExitBase::considerAddingAsFrequentExitSite(profiledCodeBlock, ExitFromDFG);
+        OSRExitBase::considerAddingAsFrequentExitSite(profiledCodeBlock, ExitFromDFG);
     }
 };