Commit 78e1b2bc authored by fpizlo@apple.com's avatar fpizlo@apple.com

CodeBlock::jettison() shouldn't call baselineVersion()

https://bugs.webkit.org/show_bug.cgi?id=123675

Reviewed by Geoffrey Garen.
        
Fix more uses of baselineVersion().

* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::jettison):
* bytecode/CodeBlock.h:
* bytecode/ProfiledCodeBlockJettisoningWatchpoint.cpp:
(JSC::ProfiledCodeBlockJettisoningWatchpoint::fireInternal):



git-svn-id: http://svn.webkit.org/repository/webkit/trunk@158507 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 1077d6ff
2013-11-02 Filip Pizlo <fpizlo@apple.com>
CodeBlock::jettison() shouldn't call baselineVersion()
https://bugs.webkit.org/show_bug.cgi?id=123675
Reviewed by Geoffrey Garen.
Fix more uses of baselineVersion().
* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::jettison):
* bytecode/CodeBlock.h:
* bytecode/ProfiledCodeBlockJettisoningWatchpoint.cpp:
(JSC::ProfiledCodeBlockJettisoningWatchpoint::fireInternal):
2013-11-02 Filip Pizlo <fpizlo@apple.com>
LLVM asserts in internal-js-tests.yaml/Octane/stress-tests/mandreel.js
......
......@@ -2851,7 +2851,9 @@ void CodeBlock::jettison(ReoptimizationMode mode)
// Count the reoptimization if that's what the user wanted.
if (mode == CountReoptimization) {
baselineVersion()->countReoptimization();
// FIXME: Maybe this should call alternative().
// https://bugs.webkit.org/show_bug.cgi?id=123677
baselineAlternative()->countReoptimization();
if (DFG::shouldShowDisassembly())
dataLog(" Did count reoptimization for ", *this, "\n");
}
......
......@@ -137,6 +137,9 @@ public:
}
CodeBlock* baselineAlternative();
// FIXME: Get rid of this.
// https://bugs.webkit.org/show_bug.cgi?id=123677
CodeBlock* baselineVersion();
void visitAggregate(SlotVisitor&);
......
......@@ -40,6 +40,8 @@ void ProfiledCodeBlockJettisoningWatchpoint::fireInternal()
m_exitKind, " at ", m_codeOrigin, "\n");
}
// FIXME: Maybe this should call alternative().
// https://bugs.webkit.org/show_bug.cgi?id=123677
CodeBlock* machineBaselineCodeBlock = m_codeBlock->baselineAlternative();
CodeBlock* sourceBaselineCodeBlock =
baselineCodeBlockForOriginAndBaselineCodeBlock(
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment