• Maya Lekova's avatar
    Revert "cppgc: Rework GC info creation" · 8b9d0138
    Maya Lekova authored
    This reverts commit d76064df.
    
    Reason for revert: Breaking MSAN - https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux%20-%20arm64%20-%20sim%20-%20MSAN/37390/overview
    
    Original change's description:
    > cppgc: Rework GC info creation
    >
    > Previously, GCInfoTrait relied on the non-trivial constructor of a
    > static object for registering a new GCInfo object. The generated code
    > is required to be thread-safe which is achieved by introducing guard
    > variables in the compiler.
    >
    > The new version is similar to Blink in that it relies on zero
    > initialization of a trivially constructible atomic.
    >
    > Compared to guard variables that are created per GCInfo registration,
    > the atomic creates less bloat (~20bytes/type) and also results in a
    > better fast path.
    >
    > Minimum example: https://godbolt.org/z/qrdTf8
    >
    > Bug: chromium:1056170
    > Change-Id: I95efbbf035b655d0440c9477f5391e310e2b71fa
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2764750
    > Reviewed-by: Omer Katz <omerkatz@chromium.org>
    > Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#73463}
    
    Bug: chromium:1056170
    Change-Id: I71960103513d6db7789d752b70727d014c2e6406
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2767020
    Auto-Submit: Maya Lekova <mslekova@chromium.org>
    Reviewed-by: 's avatarMichael Lippautz <mlippautz@chromium.org>
    Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#73466}
    8b9d0138
Name
Last commit
Last update
..
base Loading commit data...
cppgc Loading commit data...
allocation-observer-unittest.cc Loading commit data...
barrier-unittest.cc Loading commit data...
bitmap-test-utils.h Loading commit data...
bitmap-unittest.cc Loading commit data...
code-object-registry-unittest.cc Loading commit data...
embedder-tracing-unittest.cc Loading commit data...
gc-idle-time-handler-unittest.cc Loading commit data...
gc-tracer-unittest.cc Loading commit data...
heap-controller-unittest.cc Loading commit data...
heap-unittest.cc Loading commit data...
heap-utils.cc Loading commit data...
heap-utils.h Loading commit data...
index-generator-unittest.cc Loading commit data...
item-parallel-job-unittest.cc Loading commit data...
list-unittest.cc Loading commit data...
local-factory-unittest.cc Loading commit data...
local-heap-unittest.cc Loading commit data...
marking-unittest.cc Loading commit data...
marking-worklist-unittest.cc Loading commit data...
memory-reducer-unittest.cc Loading commit data...
object-start-bitmap-unittest.cc Loading commit data...
object-stats-unittest.cc Loading commit data...
persistent-handles-unittest.cc Loading commit data...
safepoint-unittest.cc Loading commit data...
slot-set-unittest.cc Loading commit data...
spaces-unittest.cc Loading commit data...
traced-reference-unittest.cc Loading commit data...
unified-heap-snapshot-unittest.cc Loading commit data...
unified-heap-unittest.cc Loading commit data...
unified-heap-utils.cc Loading commit data...
unified-heap-utils.h Loading commit data...
unmapper-unittest.cc Loading commit data...
worklist-unittest.cc Loading commit data...