On Wed, Jun 03, 2009 at 07:16:04PM +0100, Dave Korn wrote: >winsup/cygwin/ChangeLog > > * thread.cc (__cygwin_lock_lock): Delete racy optimisation. > (__cygwin_lock_unlock): Likewise. > > * winbase.h (ilockexch): Fix asm constraints. > (ilockcmpexch): Likewise. > > OK for head?
Which version of the asm constraints are these? If this is the "I'm pretty sure I know what I'm doing" version then no. If these are the versions from some tried-and-true OS/library then yes. Actually maybe this should be two checkins since the __cygwin_lock_lock stuff is uncontroversial. cgf
