* src/winsup/w32api ChangeLog include/winnt.h
[not found] ` <CAKw7uVid61wi6zeegbhhsLwN=qu0CHbuWoB+AavSzzpnAm-jsA@mail.gmail.com>
@ 2012-08-02 8:11 ` Václav Zeman
2012-08-02 9:12 ` Corinna Vinschen
0 siblings, 1 reply; 2+ messages in thread
From: Václav Zeman @ 2012-08-02 8:11 UTC (permalink / raw)
To: cygwin; +Cc: earnie
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
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: src/winsup/w32api ChangeLog include/winnt.h
2012-08-02 8:11 ` src/winsup/w32api ChangeLog include/winnt.h Václav Zeman
@ 2012-08-02 9:12 ` Corinna Vinschen
0 siblings, 0 replies; 2+ messages in thread
From: Corinna Vinschen @ 2012-08-02 9:12 UTC (permalink / raw)
To: cygwin
On Aug 2 09:56, Václav Zeman wrote:
> 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?
w32api is maintained by the Mingw32 project. Please discuss issues
on the mingw32 mailing lists, see
https://sourceforge.net/mail/?group_id=2435
Corinna
--
Corinna Vinschen Please, send mails regarding Cygwin to
Cygwin Project Co-Leader cygwin AT cygwin DOT com
Red Hat
--
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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-08-02 8:11 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20120801191737.7557.qmail@sourceware.org>
[not found] ` <CAKw7uVid61wi6zeegbhhsLwN=qu0CHbuWoB+AavSzzpnAm-jsA@mail.gmail.com>
2012-08-02 8:11 ` src/winsup/w32api ChangeLog include/winnt.h Václav Zeman
2012-08-02 9:12 ` Corinna Vinschen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).