• Clemens Backes's avatar
    [wasm] Don't block compilation on publishing · 1262eb02
    Clemens Backes authored
    If multiple compilation threads want to publish their generated code,
    they currently block on each other. This can cause multiple background
    threads to be blocked for several hundred milliseconds in the worst
    case.
    
    This CL changes this such that instead of blocking, the threads just
    put the code in a queue from where it is picked up by the thread that
    is currently publishing. Instead of blocking, the threads can then
    continue compiling more code already.
    
    This change might produce regressions, because there is now more
    TurboFan tier-up compilation happening while Liftoff code is being
    published. This might delay the completion of baseline compilation. It
    can also happen that we publish (more) TurboFan code before finishing
    baseline compilation, which would also regress compile scores.
    
    Let's see what the perf bots have to say about this CL. We might need to
    adapt certain things (like delaying TurboFan compilation until all
    Liftoff code finished), or we might just accept slight delays in Liftoff
    compilation, because tier-up will finish sooner after this CL, giving us
    peak performance earlier.
    
    R=ahaas@chromium.org
    
    Bug: v8:10330
    Change-Id: I2f5c15810a0a9fc18461f9cbf4e436ab36aa559d
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2154200
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#67308}
    1262eb02
Name
Last commit
Last update
build_overrides Loading commit data...
custom_deps Loading commit data...
docs Loading commit data...
gni Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.clang-tidy Loading commit data...
.editorconfig Loading commit data...
.flake8 Loading commit data...
.git-blame-ignore-revs Loading commit data...
.gitattributes Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.vpython Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
COMMON_OWNERS Loading commit data...
DEPS Loading commit data...
ENG_REVIEW_OWNERS Loading commit data...
INFRA_OWNERS Loading commit data...
INTL_OWNERS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
MIPS_OWNERS Loading commit data...
OWNERS Loading commit data...
PPC_OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
S390_OWNERS Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...