libev cvs ECB_MEMORY_FENCE(_RELEASE) undefined

[common at gmx.ch]

On 08/23/2011 05:45 PM, Marc Lehmann wrote:
> could you investigate why this block doesn't define it on your system? (it's
> around line 550 in ev.c):
>
>     #ifndef ECB_MEMORY_FENCE
>       #if ECB_GCC_VERSION(2,5)
>         #if __x86

__x86 is not defined here, __i386 is.

Maybe extend the check to include some more of
http://sourceforge.net/apps/mediawiki/predef/index.php?title=Architectures#Intel_x86
?

> Also, there is no really no way it can't be defined, as the block in line
> 583 defines it in all cases (but might lead to linker errors later).
583
#ifndef ECB_MEMORY_FENCE
   #if !ECB_AVOID_PTHREADS
     #define ECB_NEEDS_PTHREADS 1

863
#if ECB_MEMORY_FENCE_NEEDS_PTHREADS
# undef ECB_MEMORY_FENCE
# undef ECB_MEMORY_FENCE_ACQUIRE
# undef ECB_MEMORY_FENCE_RELEASE
#endif

> Can you cvs diff to see if its really the current cvs (there have been no
> changes for a few weeks).

I used a fresh co from yesterday.