• Tobias Tebbi's avatar
    Revert "Reland "[shared-struct] Add Atomics.Condition"" · 74d4f133
    Tobias Tebbi authored
    This reverts commit b1020a43.
    
    Reason for revert: Causes timeout for `condition-workers`: https://ci.chromium.org/ui/p/v8/builders/ci/V8%20Linux%20-%20debug/40516/overview
    
    Original change's description:
    > Reland "[shared-struct] Add Atomics.Condition"
    >
    > This is a reland of commit e2066ff6
    >
    > Changes since revert:
    > - Rebased against c9918524, which
    >   uses the external pointer table for the WaiterQueueNode stored
    >   in the state field when compressing pointers. This relaxes
    >   the alignment requirement of the state field to be 4-bytes when
    >   compressing pointers.
    > - Moved the state field into the JSSynchronizationPrimitive base
    >   class, since alignment and padding can now be made simpler.
    >
    > Original change's description:
    > > [shared-struct] Add Atomics.Condition
    > >
    > > Bug: v8:12547
    > > Change-Id: Id439aef9cab3348171a23378cdd47ede5f4d7288
    > > Cq-Include-Trybots: luci.v8.try:v8_linux_arm64_rel_ng,v8_linux64_tsan_rel_ng
    > > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3630350
    > > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org>
    > > Reviewed-by: Adam Klein <adamk@chromium.org>
    > > Commit-Queue: Shu-yu Guo <syg@chromium.org>
    > > Cr-Commit-Position: refs/heads/main@{#81734}
    >
    > Bug: v8:12547
    > Change-Id: I638304c3d5722c64bd04708ed4cf84863cdebb81
    > Cq-Include-Trybots: luci.v8.try:v8_linux_arm64_rel_ng,v8_linux64_tsan_rel_ng
    > Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3763787
    > Reviewed-by: Adam Klein <adamk@chromium.org>
    > Commit-Queue: Shu-yu Guo <syg@chromium.org>
    > Reviewed-by: Dominik Inführ <dinfuehr@chromium.org>
    > Cr-Commit-Position: refs/heads/main@{#82278}
    
    Bug: v8:12547
    Change-Id: I27c2aeb131f1b68c2240323189db88d552aa92f9
    Cq-Include-Trybots: luci.v8.try:v8_linux_arm64_rel_ng,v8_linux64_tsan_rel_ng
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3817187
    Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Auto-Submit: Tobias Tebbi <tebbi@chromium.org>
    Owners-Override: Tobias Tebbi <tebbi@chromium.org>
    Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/main@{#82292}
    74d4f133
runtime-test.cc 55.7 KB