DescriptionRevert of [SAB] Move Atomics builtins to C++ (patchset #10 id:170001 of https://codereview.chromium.org/2698813004/ )
Reason for revert:
Seems to the culprit in
BUG=chromium:695653
Original issue's description:
> [SAB] Move Atomics builtins to C++
>
> This patch refactors the Atomics builtins so that they are implemented
> as C++ builtins rather than experimental JS builtins. Previously, each
> of these functions called out to a runtime function, so no significant
> change in performance is anticipated. The goal of this patch is to
> remove the last user of experimental JS builtins so that the mechanism
> can be removed, for performance reasons. The patch includes a drive-by
> fix of a check-fail. For the most part, the patch is just moving code
> without modification from runtime-atomics.cc to
> builtins-sharedarraybuffer.cc .
>
> BUG=v8:5880
>
> Review-Url: https://codereview.chromium.org/2698813004
> Cr-Commit-Position: refs/heads/master@{#43335}
> Committed: https://chromium.googlesource.com/v8/v8/+/2b9840d86f3b51f42918b6bf1a06ff8b62b2464d
TBR=binji@chromium.org,littledan@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=v8:5880
Patch Set 1 #
Messages
Total messages: 6 (2 generated)
|