• 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
number-modulus.js 7.7 KB