• Nico Hartmann's avatar
    Revert "[wasm] Do not log code of functions whose module is not fully loaded" · 1d9c4a1f
    Nico Hartmann authored
    This reverts commit 63e243a0.
    
    Reason for revert: Speculative revert for https://ci.chromium.org/p/v8/builders/ci/V8%20Linux64%20TSAN/31811?
    
    Original change's description:
    > [wasm] Do not log code of functions whose module is not fully loaded
    > 
    > Under some circumstances, Wasm is trying to log code for which the
    > wire bytes are not fully loaded yet. This can happen during streaming
    > compilation when a few functions are already fully compiled but the
    > engine is still streaming the remaining functions.
    > 
    > If the profiler now kicks in, it will attempt to log these freshly
    > compiled functions. As these functions will not be executed before
    > the module is fully compiled, we can simply defer the logging in this
    > case.
    > 
    > R=​clemensb@chromium.org
    > 
    > Bug: chromium:1085852
    > Change-Id: Idb1061cafcba7a2a654a207402dca520f79a3bbe
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2219938
    > Reviewed-by: Clemens Backes <clemensb@chromium.org>
    > Commit-Queue: Emanuel Ziegler <ecmziegler@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#68174}
    
    TBR=clemensb@chromium.org,ecmziegler@chromium.org
    
    Change-Id: I444c70e63b876b8473bf9599643bdfe39c8f1eae
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:1085852
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2230531Reviewed-by: 's avatarNico Hartmann <nicohartmann@chromium.org>
    Commit-Queue: Nico Hartmann <nicohartmann@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#68177}
    1d9c4a1f
module-compiler.cc 118 KB