From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id B35703858D33; Wed, 22 Apr 2020 17:20:14 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org B35703858D33 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1587576014; bh=pFa3lbRBaITEtmPFrB76zyKGPU/3AHY/xjFPz5QZv1A=; h=From:To:Subject:Date:In-Reply-To:References:From; b=yqWcf9W6lF5D33GFI/0NQgfMhO3B2tvW33bfH7xyjRRBhSKrf8b87KqKFzWMJpBJd W8d26yjkkIuHAf1bArllWsZHRLiAqERk+p0YobhzZFfKPfciE0SMoBjCr3yTaQNuuN ONZ1VL13uKU1eOIKK4H2Zxcri6CriTSM0EA2NE0s= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/94622] testsuite/gcc.dg/atomic/c11-atomic-exec-1.c fails on powerpc64le with -mpcrel Date: Wed, 22 Apr 2020 17:20:14 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: target X-Bugzilla-Version: 10.0 X-Bugzilla-Keywords: wrong-code X-Bugzilla-Severity: normal X-Bugzilla-Who: cvs-commit at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: acsawdey at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/ Auto-Submitted: auto-generated MIME-Version: 1.0 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Apr 2020 17:20:14 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D94622 --- Comment #4 from CVS Commits --- The master branch has been updated by Aaron Sawdey : https://gcc.gnu.org/g:3bcdb5dec72b6d7b197821c2b814bc9fc07f4628 commit r10-7889-g3bcdb5dec72b6d7b197821c2b814bc9fc07f4628 Author: Aaron Sawdey Date: Mon Apr 20 13:01:43 2020 -0500 Use plq/pstq for atomic_{load,store} For future architecture with prefix instructions, always use plq/pstq rather than lq/stq for atomic load of quadword. Then we never have to do the doubleword swap on little endian. Before this fix, -mno-pcrel would generate lq with the doubleword swap (which was ok) and -mpcrel would generate plq, also with the doubleword swap, which was wrong. 2020-04-20 Aaron Sawdey PR target/94622 * config/rs6000/sync.md (load_quadpti): Add attr "prefixed" if TARGET_PREFIXED. (store_quadpti): Ditto. (atomic_load): Do not swap doublewords if TARGET_PREFIXED= as plq will be used and doesn't need it. (atomic_store): Ditto, for pstq.=