From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1643) id D2BDA382A2CE; Thu, 27 Oct 2022 07:50:12 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org D2BDA382A2CE DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1666857012; bh=ElBf4kp8eRZQY54RpIVngp0Kvq9+8K1jNBIg0R0XsAo=; h=From:To:Subject:Date:From; b=OdeDzPQKA4WMk+YeqRZwB9pa4qmwzsvHmMFw+zCwDG36JxpwEvN0HtCyunJGL4S/u mjlKbiCZA6FozdcpzPx6g3tmtkCGEgMV5jGRNJTcf89r0awtXeWsgV7jQGPrNTibua qPYN6XFiyPR9qgVPH03FEHWFdAURo2HDTGNrDgrs= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Thomas Schwinge To: gcc-cvs@gcc.gnu.org Subject: [gcc/devel/rust/master] Merge #1615 X-Act-Checkin: gcc X-Git-Author: bors[bot] <26634292+bors[bot]@users.noreply.github.com> X-Git-Refname: refs/heads/devel/rust/master X-Git-Oldrev: 14c942c3eb060e063fd8142d390c6d09951bf544 X-Git-Newrev: 2bc1c5309206169776bdcab862aa2b3da7fc8c95 Message-Id: <20221027075012.D2BDA382A2CE@sourceware.org> Date: Thu, 27 Oct 2022 07:50:12 +0000 (GMT) List-Id: https://gcc.gnu.org/g:2bc1c5309206169776bdcab862aa2b3da7fc8c95 commit 2bc1c5309206169776bdcab862aa2b3da7fc8c95 Merge: 14c942c3eb0 4d42744aa5d Author: bors[bot] <26634292+bors[bot]@users.noreply.github.com> Date: Wed Oct 26 08:56:37 2022 +0000 Merge #1615 1615: intrinsics: Add early implementation for atomic_store_{seqcst, relaxed, release} r=CohenArthur a=CohenArthur Needs #1614 so ignore the first commit This commit adds support for three `atomic_store_*` intrinsics declared in the core library. The mapping is as follows: - atomic_store_seqcst(dst, val) -> __atomic_store_n(dst, val, __ATOMIC_SEQ_CST) - atomic_store_relaxed(dst, val) -> __atomic_store_n(dst, val, __ATOMIC_RELAXED) - atomic_store_release(dst, val) -> __atomic_store_n(dst, val, __ATOMIC_RELEASE) The one remaining question is whether `atomic_store_unordered` can be abstracted as an atomic store with the __ATOMIC_RELAXED ordering. This commit also performs the overloading "by hand": Replacing `atomic_store_release` with `__atomic_store_4` as I cannot get the generic version to work. This will be done in future improvements. Co-authored-by: Arthur Cohen Diff: gcc/rust/backend/rust-builtins.cc | 36 +++++ gcc/rust/backend/rust-builtins.h | 2 +- gcc/rust/backend/rust-compile-intrinsic.cc | 165 ++++++++++++++++++--- gcc/testsuite/rust/compile/torture/intrinsics-4.rs | 20 +++ gcc/testsuite/rust/compile/torture/intrinsics-5.rs | 35 +++++ gcc/testsuite/rust/execute/torture/atomic_store.rs | 32 ++++ 6 files changed, 268 insertions(+), 22 deletions(-)