• Georg Neis's avatar
    Reland "[compiler] Directly read PropertyCells" · cf7cba8d
    Georg Neis authored
    This reverts commit 87df0b7e (thus
    relands 42cd9eb7), with fixes for
    the discovered issues.
    
    Original change's description:
    > Revert "[compiler] Directly read PropertyCells"
    >
    > This reverts commit 42cd9eb7.
    >
    > Reason for revert: Clusterfuzz issues, e.g.
    > https://bugs.chromium.org/p/chromium/issues/detail?id=1176318
    >
    > Original change's description:
    > > [compiler] Directly read PropertyCells
    > >
    > > Main changes:
    > >
    > > - Introduce a new broker data kind kBackgroundSerialized for objects
    > >   that can be serialized in the background (when direct reads are on).
    > >   (I'm planning to remove kPossiblyBackgroundSerialized in a followup,
    > >   in favor of a dynamic choice of kSerialized or kBackgroundSerialized).
    > > - Make PropertyCell use that new kind.
    > > - Introduce a bottleneck in runtime code for changes to PropertyCells
    > >   and make sure that a certain protocol is followed that allows
    > >   concurrent reads from the background thread.
    > > - Improve interface of PropertyCell in various ways.
    > >
    > > Bug: v8:7790
    > > Change-Id: If3d7926c3b894808811348b4b2bed153f5c06897
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2661462
    > > Reviewed-by: Toon Verwaest <verwaest@chromium.org>
    > > Reviewed-by: Ulan Degenbaev <ulan@chromium.org>
    > > Reviewed-by: Nico Hartmann <nicohartmann@chromium.org>
    > > Commit-Queue: Georg Neis <neis@chromium.org>
    > > Cr-Commit-Position: refs/heads/master@{#72586}
    >
    > TBR=ulan@chromium.org,neis@chromium.org,verwaest@chromium.org,nicohartmann@chromium.org
    >
    > Change-Id: Id04145760c49fa379bc5a3fc16eba664025a9180
    > Bug: v8:7790
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2685125
    > Reviewed-by: Georg Neis <neis@chromium.org>
    > Commit-Queue: Georg Neis <neis@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#72619}
    
    Bug: v8:7790, chromium:1176509, chromium:1176318, chromium:1176504
    Change-Id: Icaf285912bb948432a4a2d599cd174f6a5aa296e
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2685166Reviewed-by: 's avatarGeorg Neis <neis@chromium.org>
    Reviewed-by: 's avatarIgor Sheludko <ishell@chromium.org>
    Reviewed-by: 's avatarMichael Stanton <mvstanton@chromium.org>
    Reviewed-by: 's avatarUlan Degenbaev <ulan@chromium.org>
    Reviewed-by: 's avatarNico Hartmann <nicohartmann@chromium.org>
    Commit-Queue: Georg Neis <neis@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#72697}
    cf7cba8d
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...