Changeset 265136 in webkit


Ignore:
Timestamp:
Jul 31, 2020 12:45:16 AM (4 years ago)
Author:
ysuzuki@apple.com
Message:

[JSC] Follow-up changes after r265036
https://bugs.webkit.org/show_bug.cgi?id=214982

Reviewed by Darin Adler.

  • dfg/DFGOSRExit.cpp:

(JSC::DFG::OSRExit::compileExit): Remove dupe definitions in OSRExit.

  • jit/JITCall32_64.cpp:

(JSC::JIT::emit_op_iterator_open): We should use emitJumpSlowCaseIfNotJSCell(regT1).

Location:
trunk/Source/JavaScriptCore
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r265122 r265136  
     12020-07-31  Yusuke Suzuki  <ysuzuki@apple.com>
     2
     3        [JSC] Follow-up changes after r265036
     4        https://bugs.webkit.org/show_bug.cgi?id=214982
     5
     6        Reviewed by Darin Adler.
     7
     8        * dfg/DFGOSRExit.cpp:
     9        (JSC::DFG::OSRExit::compileExit): Remove dupe definitions in OSRExit.
     10        * jit/JITCall32_64.cpp:
     11        (JSC::JIT::emit_op_iterator_open): We should use emitJumpSlowCaseIfNotJSCell(regT1).
     12
    1132020-07-30  Keith Miller  <keith_miller@apple.com>
    214
  • trunk/Source/JavaScriptCore/dfg/DFGOSRExit.cpp

    r265036 r265136  
    604604                        sideState->tmps[i] = recovery.constant();
    605605                        break;
    606                        
    607 #if USE(JSVALUE64)
     606
    608607                    case UnboxedInt32InGPR:
    609608                    case Int32DisplacedInJSStack: {
     
    612611                    }
    613612
     613                    case UnboxedBooleanInGPR: {
     614                        sideState->tmps[i] = jsBoolean(static_cast<bool>(tmpScratch[i + tmpOffset]));
     615                        break;
     616                    }
     617
     618#if USE(JSVALUE64)
    614619                    case BooleanDisplacedInJSStack:
    615620                    case CellDisplacedInJSStack:
     
    620625                        break;
    621626                    }
    622 
    623                     case UnboxedBooleanInGPR: {
    624                         sideState->tmps[i] = jsBoolean(static_cast<bool>(tmpScratch[i + tmpOffset]));
    625                         break;
    626                     }
    627 
    628627#else // USE(JSVALUE32_64)
    629                     case UnboxedInt32InGPR:
    630                     case Int32DisplacedInJSStack: {
    631                         sideState->tmps[i] = jsNumber(static_cast<int32_t>(tmpScratch[i + tmpOffset]));
    632                         break;
    633                     }
    634 
    635628                    case InPair:
    636629                    case DisplacedInJSStack: {
     
    646639                    }
    647640
    648                     case BooleanDisplacedInJSStack:
    649                     case UnboxedBooleanInGPR: {
     641                    case BooleanDisplacedInJSStack: {
    650642                        sideState->tmps[i] = jsBoolean(static_cast<bool>(tmpScratch[i + tmpOffset]));
    651643                        break;
    652644                    }
    653                        
    654645#endif // USE(JSVALUE64)
    655646
    656                     default: 
     647                    default:
    657648                        RELEASE_ASSERT_NOT_REACHED();
    658649                        break;
  • trunk/Source/JavaScriptCore/jit/JITCall32_64.cpp

    r265036 r265136  
    390390    compileOpCall<OpIteratorOpen>(instruction, m_callLinkInfoIndex++);
    391391
    392     VirtualRegister vr = destinationFor(bytecode, m_bytecodeIndex.checkpoint()).virtualRegister();
    393392    advanceToNextCheckpoint();
    394393   
     
    396395    const Identifier* ident = &vm().propertyNames->next;
    397396   
    398     emitJumpSlowCaseIfNotJSCell(vr, regT1);
     397    emitJumpSlowCaseIfNotJSCell(regT1);
    399398
    400399    GPRReg tagIteratorGPR = regT1;
Note: See TracChangeset for help on using the changeset viewer.