Skip to content
  • fpizlo@apple.com's avatar
    Variable event stream (for DFG OSR exit) should be explicit about where on the... · ee4d8a7a
    fpizlo@apple.com authored
    Variable event stream (for DFG OSR exit) should be explicit about where on the stack a SetLocal put a value
    https://bugs.webkit.org/show_bug.cgi?id=122178
    
    Reviewed by Geoffrey Garen.
            
    Now if the DFG stores the value of a variable into the stack explicitly via a SetLocal,
    it will record where on the stack it stored the value in addition to recording where on
    the stack the bytecode would have done the SetLocal. Previously it just recorded the
    format and the bytecode variable. Recording just the bytecode variable is currently fine
    since the DFG always executes SetLocal's to the same stack location that the bytecode
    would have used. But that prevents stack compression (webkit.org/b/122024) so this patch
    allows the SetLocal to say both the bytecode variable that we're speaking of and the
    actual stack location to which the SetLocal stored the value.
            
    This had to touch a lot of code, so I took the opportunity to also resolve
    webkit.org/b/108019.
    
    * bytecode/Operands.h:
    (JSC::Operands::hasOperand):
    * dfg/DFGFlushFormat.h:
    (JSC::DFG::dataFormatFor):
    * dfg/DFGMinifiedID.h:
    (JSC::DFG::MinifiedID::bits):
    (JSC::DFG::MinifiedID::invalidID):
    (JSC::DFG::MinifiedID::otherInvalidID):
    * dfg/DFGSpeculativeJIT.cpp:
    (JSC::DFG::SpeculativeJIT::compileMovHint):
    (JSC::DFG::SpeculativeJIT::compileInlineStart):
    (JSC::DFG::SpeculativeJIT::compileCurrentBlock):
    * dfg/DFGSpeculativeJIT.h:
    (JSC::DFG::SpeculativeJIT::recordSetLocal):
    * dfg/DFGSpeculativeJIT32_64.cpp:
    (JSC::DFG::SpeculativeJIT::compile):
    * dfg/DFGSpeculativeJIT64.cpp:
    (JSC::DFG::SpeculativeJIT::compile):
    * dfg/DFGValueSource.cpp:
    (JSC::DFG::ValueSource::dump):
    * dfg/DFGValueSource.h:
    (JSC::DFG::ValueSource::ValueSource):
    (JSC::DFG::ValueSource::forFlushFormat):
    (JSC::DFG::ValueSource::forDataFormat):
    (JSC::DFG::ValueSource::isSet):
    (JSC::DFG::ValueSource::kind):
    (JSC::DFG::ValueSource::valueRecovery):
    (JSC::DFG::ValueSource::id):
    (JSC::DFG::ValueSource::virtualRegister):
    * dfg/DFGVariableEvent.cpp:
    (JSC::DFG::VariableEvent::dump):
    (JSC::DFG::VariableEvent::dumpSpillInfo):
    * dfg/DFGVariableEvent.h:
    (JSC::DFG::VariableEvent::fillGPR):
    (JSC::DFG::VariableEvent::fillPair):
    (JSC::DFG::VariableEvent::fillFPR):
    (JSC::DFG::VariableEvent::spill):
    (JSC::DFG::VariableEvent::death):
    (JSC::DFG::VariableEvent::setLocal):
    (JSC::DFG::VariableEvent::movHint):
    (JSC::DFG::VariableEvent::id):
    (JSC::DFG::VariableEvent::gpr):
    (JSC::DFG::VariableEvent::tagGPR):
    (JSC::DFG::VariableEvent::payloadGPR):
    (JSC::DFG::VariableEvent::fpr):
    (JSC::DFG::VariableEvent::spillRegister):
    (JSC::DFG::VariableEvent::bytecodeRegister):
    (JSC::DFG::VariableEvent::machineRegister):
    (JSC::DFG::VariableEvent::variableRepresentation):
    * dfg/DFGVariableEventStream.cpp:
    (JSC::DFG::VariableEventStream::reconstruct):
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@156747 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    ee4d8a7a