⚠ Archived content — this site is no longer maintained.   Current WebKit documentation is at docs.webkit.org.

Changeset 271464 in webkit


Ignore:
Timestamp:
Jan 13, 2021, 3:18:33 PM (6 years ago)
Author:
Russell Epstein
Message:

Revert r270665. rdar://problem/73165685

Location:
branches/safari-611-branch
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • branches/safari-611-branch/PerformanceTests/APIBench/api-bench

    r270665 r271464  
    3838UPCOMING_API=()
    3939ARCHS=${ARCHS:-$(uname -m)}
    40 CONFIGURATION=${CONFIGURATION:-Release}
    4140
    4241shouldBuild=true
    … …  
    150149
    151150    log "Building $BENCHMARK_NAME... (FRAMEWORK_SEARCH_PATHS='$BUILD_DIRECTORY')"
    152     xcodebuild -project "$project" -target "$BENCHMARK_NAME" -configuration "$CONFIGURATION" build "FRAMEWORK_SEARCH_PATHS='$BUILD_DIRECTORY'" "ARCHS='$ARCHS'"
     151    xcodebuild -project "$project" -target "$BENCHMARK_NAME" -configuration Release build "FRAMEWORK_SEARCH_PATHS='$BUILD_DIRECTORY'" "ARCHS='$ARCHS'"
    153152}
    154153
    … …  
    163162
    164163runNativeBenchmark() {
    165     pushd "build/$CONFIGURATION" &> /dev/null
     164    pushd "build/Release" &> /dev/null
    166165    "./$BENCHMARK_NAME"
    167     local exitCode=$?
    168     if [ $exitCode -ne 0 ]; then
    169         echo "Benchmark exited with an error: $PWD/$BENCHMARK_NAME exited with $exitCode" 1>&2
    170     fi
    171166    popd &> /dev/null
    172     return $exitCode
    173167}
    174168
    … …  
    186180    local result
    187181    result=$(runBenchmarkIteration)
    188     if [ $? -ne 0 ]; then
    189         exit 1
    190     fi
    191182    RESULTS[$((BENCHMARK_ID * iterations + ITERATION))]=$result
    192183    log "Finished in ${result}ms"

  • branches/safari-611-branch/PerformanceTests/ChangeLog

    r271126 r271464  
     12021-01-13  Russell Epstein  <repstein@apple.com>
     2
     3        Revert r270665. rdar://problem/73165685
     4
    152021-01-03  Beth Dakin  <bdakin@apple.com>
    26

  • branches/safari-611-branch/Source/JavaScriptCore/API/APICast.h

    r270665 r271464  
    8989}
    9090
    91 #if CPU(ADDRESS64)
    92 inline JSC::JSValue toJS(JSValueRef value)
    93 {
    94     return bitwise_cast<JSC::JSValue>(value);
    95 }
    96 #endif
    97 
    9891inline JSC::JSValue toJSForGC(JSC::JSGlobalObject* globalObject, JSValueRef v)
    9992{
    … …  
    156149}
    157150
    158 #if CPU(ADDRESS64)
    159 inline JSValueRef toRef(JSC::JSValue v)
    160 {
    161     return bitwise_cast<JSValueRef>(v);
    162 }
    163 #endif
    164 
    165151inline JSObjectRef toRef(JSC::JSObject* o)
    166152{

  • branches/safari-611-branch/Source/JavaScriptCore/API/JSValue.mm

    r270665 r271464  
    399399- (BOOL)isUndefined
    400400{
    401 #if !CPU(ADDRESS64)
    402401    return JSValueIsUndefined([_context JSGlobalContextRef], m_value);
    403 #else
    404     return toJS(m_value).isUndefined();
    405 #endif
    406402}
    407403
    408404- (BOOL)isNull
    409405{
    410 #if !CPU(ADDRESS64)
    411     return JSValueIsUndefined([_context JSGlobalContextRef], m_value);
    412 #else
    413     return toJS(m_value).isNull();
    414 #endif
     406    return JSValueIsNull([_context JSGlobalContextRef], m_value);
    415407}
    416408
    417409- (BOOL)isBoolean
    418410{
    419 #if !CPU(ADDRESS64)
    420     return JSValueIsUndefined([_context JSGlobalContextRef], m_value);
    421 #else
    422     return toJS(m_value).isBoolean();
    423 #endif
     411    return JSValueIsBoolean([_context JSGlobalContextRef], m_value);
    424412}
    425413
    426414- (BOOL)isNumber
    427415{
    428 #if !CPU(ADDRESS64)
    429     return JSValueIsUndefined([_context JSGlobalContextRef], m_value);
    430 #else
    431     return toJS(m_value).isNumber();
    432 #endif
     416    return JSValueIsNumber([_context JSGlobalContextRef], m_value);
    433417}
    434418
    435419- (BOOL)isString
    436420{
    437 #if !CPU(ADDRESS64)
    438     return JSValueIsUndefined([_context JSGlobalContextRef], m_value);
    439 #else
    440     return toJS(m_value).isString();
    441 #endif
     421    return JSValueIsString([_context JSGlobalContextRef], m_value);
    442422}
    443423
    444424- (BOOL)isObject
    445425{
    446 #if !CPU(ADDRESS64)
    447426    return JSValueIsObject([_context JSGlobalContextRef], m_value);
    448 #else
    449     return toJS(m_value).isObject();
    450 #endif
    451427}
    452428
    453429- (BOOL)isSymbol
    454430{
    455 #if !CPU(ADDRESS64)
    456431    return JSValueIsSymbol([_context JSGlobalContextRef], m_value);
    457 #else
    458     return toJS(m_value).isSymbol();
    459 #endif
    460432}
    461433

  • branches/safari-611-branch/Source/JavaScriptCore/API/JSValueRef.cpp

    r271463 r271464  
    5555        return kJSTypeUndefined;
    5656    }
    57 #if !CPU(ADDRESS64)
    58     JSGlobalObject* globalObject = toJS(ctx);
    59     JSLockHolder locker(globalObject);
    60     JSValue jsValue = toJS(globalObject, value);
    61 #else
    62     JSValue jsValue = toJS(value);
    63 #endif
     57    JSGlobalObject* globalObject = toJS(ctx);
     58    JSLockHolder locker(globalObject);
     59
     60    JSValue jsValue = toJS(globalObject, value);
    6461
    6562    if (jsValue.isUndefined())
    … …  
    8582        return false;
    8683    }
    87 #if !CPU(ADDRESS64)
    88     JSGlobalObject* globalObject = toJS(ctx);
    89     JSLockHolder locker(globalObject);
     84    JSGlobalObject* globalObject = toJS(ctx);
     85    JSLockHolder locker(globalObject);
     86
    9087    return toJS(globalObject, value).isUndefined();
    91 #else
    92     return toJS(value).isUndefined();
    93 #endif
    9488}
    9589
    … …  
    10094        return false;
    10195    }
    102 
    103 #if !CPU(ADDRESS64)
    104     JSGlobalObject* globalObject = toJS(ctx);
    105     JSLockHolder locker(globalObject);
     96    JSGlobalObject* globalObject = toJS(ctx);
     97    JSLockHolder locker(globalObject);
     98
    10699    return toJS(globalObject, value).isNull();
    107 #else
    108     return toJS(value).isNull();
    109 #endif
    110100}
    111101
    … …  
    116106        return false;
    117107    }
    118 #if !CPU(ADDRESS64)
    119     JSGlobalObject* globalObject = toJS(ctx);
    120     JSLockHolder locker(globalObject);
     108    JSGlobalObject* globalObject = toJS(ctx);
     109    JSLockHolder locker(globalObject);
     110
    121111    return toJS(globalObject, value).isBoolean();
    122 #else
    123     return toJS(value).isBoolean();
    124 #endif
    125112}
    126113
    … …  
    131118        return false;
    132119    }
    133 #if !CPU(ADDRESS64)
    134     JSGlobalObject* globalObject = toJS(ctx);
    135     JSLockHolder locker(globalObject);
     120    JSGlobalObject* globalObject = toJS(ctx);
     121    JSLockHolder locker(globalObject);
     122
    136123    return toJS(globalObject, value).isNumber();
    137 #else
    138     return toJS(value).isNumber();
    139 #endif
    140124}
    141125
    … …  
    146130        return false;
    147131    }
    148 #if !CPU(ADDRESS64)
    149     JSGlobalObject* globalObject = toJS(ctx);
    150     JSLockHolder locker(globalObject);
     132    JSGlobalObject* globalObject = toJS(ctx);
     133    JSLockHolder locker(globalObject);
     134
    151135    return toJS(globalObject, value).isString();
    152 #else
    153     return toJS(value).isString();
    154 #endif
    155136}
    156137
    … …  
    161142        return false;
    162143    }
    163 #if !CPU(ADDRESS64)
    164     JSGlobalObject* globalObject = toJS(ctx);
    165     JSLockHolder locker(globalObject);
     144    JSGlobalObject* globalObject = toJS(ctx);
     145    JSLockHolder locker(globalObject);
     146
    166147    return toJS(globalObject, value).isObject();
    167 #else
    168     return toJS(value).isObject();
    169 #endif
    170148}
    171149
    … …  
    176154        return false;
    177155    }
    178 #if !CPU(ADDRESS64)
    179     JSGlobalObject* globalObject = toJS(ctx);
    180     JSLockHolder locker(globalObject);
     156    JSGlobalObject* globalObject = toJS(ctx);
     157    JSLockHolder locker(globalObject);
     158
    181159    return toJS(globalObject, value).isSymbol();
    182 #else
    183     return toJS(value).isSymbol();
    184 #endif
    185160}
    186161
    … …  
    303278        return nullptr;
    304279    }
    305 #if !CPU(ADDRESS64)
    306     JSGlobalObject* globalObject = toJS(ctx);
    307     JSLockHolder locker(globalObject);
     280    JSGlobalObject* globalObject = toJS(ctx);
     281    JSLockHolder locker(globalObject);
     282
    308283    return toRef(globalObject, jsUndefined());
    309 #else
    310     return toRef(jsUndefined());
    311 #endif
    312284}
    313285
    … …  
    318290        return nullptr;
    319291    }
    320 #if !CPU(ADDRESS64)
    321     JSGlobalObject* globalObject = toJS(ctx);
    322     JSLockHolder locker(globalObject);
     292    JSGlobalObject* globalObject = toJS(ctx);
     293    JSLockHolder locker(globalObject);
     294
    323295    return toRef(globalObject, jsNull());
    324 #else
    325     return toRef(jsNull());
    326 #endif
    327296}
    328297
    … …  
    333302        return nullptr;
    334303    }
    335 #if !CPU(ADDRESS64)
    336     JSGlobalObject* globalObject = toJS(ctx);
    337     JSLockHolder locker(globalObject);
     304    JSGlobalObject* globalObject = toJS(ctx);
     305    JSLockHolder locker(globalObject);
     306
    338307    return toRef(globalObject, jsBoolean(value));
    339 #else
    340     return toRef(jsBoolean(value));
    341 #endif
    342308}
    343309
    … …  
    348314        return nullptr;
    349315    }
    350 #if !CPU(ADDRESS64)
    351     JSGlobalObject* globalObject = toJS(ctx);
    352     JSLockHolder locker(globalObject);
     316    JSGlobalObject* globalObject = toJS(ctx);
     317    JSLockHolder locker(globalObject);
     318
    353319    return toRef(globalObject, jsNumber(purifyNaN(value)));
    354 #else
    355     return toRef(jsNumber(purifyNaN(value)));
    356 #endif
    357320}
    358321

  • branches/safari-611-branch/Source/JavaScriptCore/ChangeLog

    r271463 r271464  
     12021-01-13  Russell Epstein  <repstein@apple.com>
     2
     3        Revert r270665. rdar://problem/73165685
     4
    152021-01-13  Russell Epstein  <repstein@apple.com>
    26
Note: See TracChangeset for help on using the changeset viewer.