Changeset 165869 in webkit for trunk/Source/JavaScriptCore/dfg/DFGBasicBlock.cpp

Timestamp:

Mar 18, 2014, 9:44:22 PM (11 years ago)

Author:

[email protected]

Message:

Get rid of Flush in SSA
​https://bugs.webkit.org/show_bug.cgi?id=130440

Reviewed by Sam Weinig.

This is basically a red patch. We used to use backwards flow for determining what was
flushed, until it became clear that this doesn't make sense. Now the Flush nodes don't
accomplish anything. Keeping them around in SSA can only make things hard.

• CMakeLists.txt:
• GNUmakefile.list.am:
• JavaScriptCore.vcxproj/JavaScriptCore.vcxproj:
• JavaScriptCore.xcodeproj/project.pbxproj:
• dfg/DFGBasicBlock.cpp:

(JSC::DFG::BasicBlock::SSAData::SSAData):

• dfg/DFGBasicBlock.h:
• dfg/DFGFlushLivenessAnalysisPhase.cpp: Removed.
• dfg/DFGFlushLivenessAnalysisPhase.h: Removed.
• dfg/DFGGraph.cpp:

(JSC::DFG::Graph::dump):

• dfg/DFGPlan.cpp:

(JSC::DFG::Plan::compileInThreadImpl):

• dfg/DFGSSAConversionPhase.cpp:

(JSC::DFG::SSAConversionPhase::run):

• ftl/FTLLowerDFGToLLVM.cpp:

(JSC::FTL::LowerDFGToLLVM::compileNode):

File:

• trunk/Source/JavaScriptCore/dfg/DFGBasicBlock.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/dfg/DFGBasicBlock.cpp

@@ -112 +112 @@
 
 BasicBlock::SSAData::SSAData(BasicBlock* block)
-    : flushAtHead(OperandsLike, block->variablesAtHead)
-    , flushAtTail(OperandsLike, block->variablesAtHead)
-    , availabilityAtHead(OperandsLike, block->variablesAtHead)
+    : availabilityAtHead(OperandsLike, block->variablesAtHead)
     , availabilityAtTail(OperandsLike, block->variablesAtHead)
 {