• Leszek Swirski's avatar
    Revert "Reland "[heap] Refactor atomic marking phase"" · 3d3d9c50
    Leszek Swirski authored
    This reverts commit 25e32252.
    
    Reason for revert: Suspect for roll failure: https://ci.chromium.org/ui/p/chromium/builders/try/android_optional_gpu_tests_rel/98554/overview
    
    Original change's description:
    > Reland "[heap] Refactor atomic marking phase"
    >
    > This is a reland of commit a3f66927
    >
    > The reland addresses a few CHECKs that were too agressive and also
    > properly adjusts Oilpan's marking configurations depending on V8's
    > flags.
    >
    > Original change's description:
    > > [heap] Refactor atomic marking phase
    > >
    > > The atomic marking phase was organized in many distinct smaller
    > > phases. In particular, before http://crrev.com/c/3584115 the marking
    > > phase split into two large separate phases.
    > >
    > > This CL reorganizes marking into two phases that perform regular V8
    > > heap marking, Oilpan, and ephemerons:
    > > - A parallel phase that likely drains all marking worklists;
    > > - A single-threaded final phase to catch any left overs;
    > >
    > > This avoids artificial splitting in phases and also avoids repeated
    > > starting and joining of jobs.
    > >
    > > Change-Id: I5cccfc5777837d9ece10d8f4925781bf2d07d9da
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3602507
    > > Reviewed-by: Omer Katz <omerkatz@chromium.org>
    > > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org>
    > > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > > Cr-Commit-Position: refs/heads/main@{#80265}
    >
    > Change-Id: I26648da361b92d787c173aa9d390100ce8958728
    > Bug: chromium:1320896
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3616519
    > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > Reviewed-by: Omer Katz <omerkatz@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#80301}
    
    Bug: chromium:1320896
    Change-Id: I01742f25d54de8e4e22fefe87ce61ba295950baa
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3620286
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Owners-Override: Leszek Swirski <leszeks@chromium.org>
    Auto-Submit: Leszek Swirski <leszeks@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#80311}
    3d3d9c50
concurrent-marker.h 2.52 KB