• Jakob Gruber's avatar
    Revert "[compiler] Don't remove OSR code cache if deoptimizing at out of loop" · 87f80671
    Jakob Gruber authored
    This reverts commit 190b5d95.
    
    Reason for revert: We should understand & fix regressions, see crbug.com/1304870#c9.
    
    Original change's description:
    > [compiler] Don't remove OSR code cache if deoptimizing at out of loop
    >
    > The main purpose of OSR compilation is fasten inner loop execution, the
    > OSR code cache is still correct for loop if optimizing at out of loop,
    > keep OSR code cache can reduce unnecessary slow bytecode execution with
    > feedback collection and avoid re-OSR compilation.
    > This CL can improve JetStream2 case navier-stokes by ~6%.
    >
    > Change-Id: I9518317fb922071b131cab5b56998a0fc198804a
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3494981
    > Reviewed-by: Leszek Swirski <leszeks@chromium.org>
    > Reviewed-by: Jakob Gruber <jgruber@chromium.org>
    > Commit-Queue: Tao Pan <tao.pan@intel.com>
    > Cr-Commit-Position: refs/heads/main@{#79413}
    
    Bug: chromium:1304870
    Change-Id: I8791edc34b66ef9dd0b477d3e340e85b0617ef59
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3515732
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Reviewed-by: 's avatarJakob Gruber <jgruber@chromium.org>
    Reviewed-by: 's avatarToon Verwaest <verwaest@chromium.org>
    Commit-Queue: Jakob Gruber <jgruber@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#79435}
    87f80671
deoptimizer.cc 83.1 KB