• Jakob Gruber's avatar
    Remove FlagScope use in test-js-to-wasm.cc · a8a8e6b7
    Jakob Gruber authored
    Flag reads from background threads are unfortunately scattered and
    hard to completely avoid in the current state of V8. An example TSAN
    failure:
    
    ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux64%20TSAN/36369/overview
    
    The root cause is that FlagScope destruction modifies flag values on
    the main thread, racing with flag reads from the background thread. In
    cctests, there's no need to reset flags back to initial values at the
    end of tests. Let's simply remove the problematic flag scopes.
    
    Bug: v8:11658
    Change-Id: I59ed3794ddc9ed570772726a423dc22afc4dc207
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2843346
    Auto-Submit: Jakob Gruber <jgruber@chromium.org>
    Reviewed-by: 's avatarClemens Backes <clemensb@chromium.org>
    Commit-Queue: Clemens Backes <clemensb@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#74082}
    a8a8e6b7
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
debugging Loading commit data...
fuzzer Loading commit data...
fuzzilli 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...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...