X-Recipient: archive-cygwin@delorie.com
X-SWARE-Spam-Status: No, hits=-5.3 required=5.0	tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,KHOP_RCVD_TRUST,KHOP_THREADED,RCVD_IN_DNSWL_LOW,RCVD_IN_HOSTKARMA_YE
X-Spam-Check-By: sourceware.org
MIME-Version: 1.0
In-Reply-To: <CAKw7uVid61wi6zeegbhhsLwN=qu0CHbuWoB+AavSzzpnAm-jsA@mail.gmail.com>
References: <20120801191737.7557.qmail@sourceware.org>	<CAKw7uVid61wi6zeegbhhsLwN=qu0CHbuWoB+AavSzzpnAm-jsA@mail.gmail.com>
Date: Thu, 2 Aug 2012 09:56:26 +0200
Message-ID: <CAKw7uVjpWiQOhKOkpGUx6dPCaQO3xVH75Re1Oyp+vzt9YKX6QQ@mail.gmail.com>
Subject: src/winsup/w32api ChangeLog include/winnt.h
From: =?UTF-8?Q?V=C3=A1clav_Zeman?= <vhaisman@gmail.com>
To: cygwin@cygwin.com
Cc: earnie@cygwin.com
Content-Type: text/plain; charset=UTF-8
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie.com@cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com

On 1 August 2012 21:17,  <earnie@...> wrote:
> CVSROOT:        /cvs/src
> Module name:    src
> Changes by:     earnie@...   2012-08-01 19:17:37
>
> Modified files:
>         winsup/w32api  : ChangeLog
>         winsup/w32api/include: winnt.h
>
> Log message:
>         * include/winnt.h (MemoryBarrier): Add definition.
>
> Patches:
> http://sourceware.org/cgi-bin/cvsweb.cgi/src/winsup/w32api/ChangeLog.diff?cvsroot=src&r1=1.1121&r2=1.1122
> http://sourceware.org/cgi-bin/cvsweb.cgi/src/winsup/w32api/include/winnt.h.diff?cvsroot=src&r1=1.139&r2=1.140
>

+# else
+   FORCEINLINE VOID MemoryBarrier (VOID) {
+       LONG Barrier = 0;
+       __asm__ __volatile__("xchgl %%eax,%0 "
+         :"=r" (Barrier));
+   }

Since this is for GCC anyway, why not using __sync_synchronize() in
the body of the function instead?

--
VZ

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

