Skip to content
  • fpizlo@apple.com's avatar
    VariableAccessData::flushFormat() should be the universal way of deciding how... · 571d3b24
    fpizlo@apple.com authored
    VariableAccessData::flushFormat() should be the universal way of deciding how to speculate on stores to locals and how locals are formatted
    https://bugs.webkit.org/show_bug.cgi?id=121142
    
    Reviewed by Geoffrey Garen.
            
    Make everyone rely on VariableAccessData::flushFormat() instead of trying to
    compute that information from scratch. The FTL already used flushFormat(), now
    the DFG does, too.
    
    * dfg/DFGArgumentPosition.h:
    (JSC::DFG::ArgumentPosition::someVariable):
    (JSC::DFG::ArgumentPosition::flushFormat):
    * dfg/DFGCSEPhase.cpp:
    (JSC::DFG::CSEPhase::performNodeCSE):
    * dfg/DFGFixupPhase.cpp:
    (JSC::DFG::FixupPhase::fixupSetLocalsInBlock):
    * dfg/DFGGraph.cpp:
    (JSC::DFG::Graph::dump):
    * dfg/DFGInPlaceAbstractState.cpp:
    (JSC::DFG::InPlaceAbstractState::mergeStateAtTail):
    * dfg/DFGJITCompiler.h:
    (JSC::DFG::JITCompiler::noticeOSREntry):
    * dfg/DFGSpeculativeJIT.cpp:
    (JSC::DFG::SpeculativeJIT::compileInlineStart):
    (JSC::DFG::SpeculativeJIT::compileCurrentBlock):
    (JSC::DFG::SpeculativeJIT::checkArgumentTypes):
    * dfg/DFGSpeculativeJIT32_64.cpp:
    (JSC::DFG::SpeculativeJIT::compile):
    * dfg/DFGSpeculativeJIT64.cpp:
    (JSC::DFG::SpeculativeJIT::compile):
    * dfg/DFGValueSource.h:
    (JSC::DFG::ValueSource::forFlushFormat):
    * dfg/DFGVariableAccessDataDump.cpp:
    (JSC::DFG::VariableAccessDataDump::dump):
    * ftl/FTLLowerDFGToLLVM.cpp:
    (JSC::FTL::LowerDFGToLLVM::compileSetLocal):
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@155564 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    571d3b24