• Benedikt Meurer's avatar
    [turbofan] Propagate kIdentifyZeros correctly for modulus. · c4ada3de
    Benedikt Meurer authored
    For NumberModulus and SpeculativeNumberModulus there's no observable
    difference between 0 and -0 for the right hand side, since both of them
    result in NaN (in general the sign of the right hand side is ignored
    for modulus in JavaScript). For the left hand side we can just propagate
    the zero identification part of the truncation, since we only care about
    -0 on the left hand side if the use nodes care about -0 too.
    
    This further improves the Kraken/audio-oscillator test from around 67ms
    to 64ms.
    
    Bug: v8:8015, v8:8178
    Change-Id: I1f51d42f7df08aaa28a9b0ddd3177df6b76be98c
    Reviewed-on: https://chromium-review.googlesource.com/c/1260024
    Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
    Reviewed-by: 's avatarSigurd Schneider <sigurds@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#56372}
    c4ada3de
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...