From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 8754 invoked by alias); 8 Nov 2011 19:54:43 -0000 Received: (qmail 8740 invoked by uid 22791); 8 Nov 2011 19:54:42 -0000 X-SWARE-Spam-Status: No, hits=-2.4 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,TW_CX X-Spam-Check-By: sourceware.org Received: from ra.se.axis.com (HELO ra.se.axis.com) (195.60.68.13) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 08 Nov 2011 19:54:20 +0000 Received: from localhost (localhost [127.0.0.1]) by ra.se.axis.com (Postfix) with ESMTP id 91E534B35C; Tue, 8 Nov 2011 20:54:13 +0100 (CET) Received: from ra.se.axis.com ([127.0.0.1]) by localhost (ra.se.axis.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id pD65BfN3563p; Tue, 8 Nov 2011 20:54:12 +0100 (CET) Received: from thoth.se.axis.com (thoth.se.axis.com [10.0.2.173]) by ra.se.axis.com (Postfix) with ESMTP id BDB3E4B188; Tue, 8 Nov 2011 20:54:12 +0100 (CET) Received: from ignucius.se.axis.com (ignucius.se.axis.com [10.88.21.50]) by thoth.se.axis.com (Postfix) with ESMTP id BA29934061; Tue, 8 Nov 2011 20:54:12 +0100 (CET) Received: from ignucius.se.axis.com (localhost [127.0.0.1]) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) with ESMTP id pA8JsCF6020898; Tue, 8 Nov 2011 20:54:12 +0100 Received: (from hp@localhost) by ignucius.se.axis.com (8.12.8p1/8.12.8/Debian-2woody1) id pA8JsCkW020894; Tue, 8 Nov 2011 20:54:12 +0100 Date: Tue, 08 Nov 2011 20:22:00 -0000 Message-Id: <201111081954.pA8JsCkW020894@ignucius.se.axis.com> From: Hans-Peter Nilsson To: amacleod@redhat.com CC: hp@axis.com, hp@axis.com, gcc-patches@gcc.gnu.org, libstdc++@gcc.gnu.org, bkoz@redhat.com, rth@redhat.com In-reply-to: <4EB7EEDF.1040103@redhat.com> (message from Andrew MacLeod on Mon, 7 Nov 2011 15:44:47 +0100) Subject: Re: cxx-mem-model merge [6 of 9] - libstdc++-v3 MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=iso-8859-1 Content-Transfer-Encoding: 8BIT Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2011-11/txt/msg01240.txt.bz2 > From: Andrew MacLeod > Date: Mon, 7 Nov 2011 15:44:47 +0100 > >> On 11/06/2011 07:38 PM, Hans-Peter Nilsson wrote: > >>> This (formally a change in the range 181027:181034) got me three > >>> libstdc++ regressions for cris-elf, > >> yes, I have a final pending patch which didn't make it to the branch > >> before the merge. It changes the behaviour of atomic_flag on targets > >> with no compare_and_swap. I *think* it will resolve your problem. > >> > >> I've attached the early version of the patch which you can try. Its > >> missing a documentation change I was going to add tomorrow before > >> submitting, but we can see if it resolves your problem. Give it a shot > >> and let me know. > > Sorry, no it didn't help. No additional regressions either > > though, but the undef'd reference changed to two: > I should also change it so that the compiler issues an error if > atomic_thread_fence doesn't resolve to a something since that implies > there is no __sync_sychronize() either, which is also minimally expected > at this point. It should never become an external reference. JFTR, you already know this; I saw some commits go by that I thought was addressing this, but didn't, and then I realized it'd be better to track it using bugzilla, so I entered PR51038. brgds, H-P