compile error in Windows CE 6.0(interlocked)

[Akira Takahashi <faithandbrave@…>]

boost/detail/interlocked.hpp is compile error in Windows CE platform.

error C2733 : second C linkage of overloaded function 'InterlockedXXX' not allowed

I correct follow change:

before

#elif defined(_WIN32_WCE)

// under Windows CE we still have old-style Interlocked* functions

extern "C" long __cdecl InterlockedIncrement( long* );
extern "C" long __cdecl InterlockedDecrement( long* );
extern "C" long __cdecl InterlockedCompareExchange( long*, long, long );
extern "C" long __cdecl InterlockedExchange( long*, long );
extern "C" long __cdecl InterlockedExchangeAdd( long*, long );

# define BOOST_INTERLOCKED_INCREMENT InterlockedIncrement
# define BOOST_INTERLOCKED_DECREMENT InterlockedDecrement
# define BOOST_INTERLOCKED_COMPARE_EXCHANGE InterlockedCompareExchange
# define BOOST_INTERLOCKED_EXCHANGE InterlockedExchange
# define BOOST_INTERLOCKED_EXCHANGE_ADD InterlockedExchangeAdd

after:

# define BOOST_INTERLOCKED_INCREMENT InterlockedIncrement
# define BOOST_INTERLOCKED_DECREMENT InterlockedDecrement
# define BOOST_INTERLOCKED_COMPARE_EXCHANGE InterlockedCompareExchange
# define BOOST_INTERLOCKED_EXCHANGE InterlockedExchange
# define BOOST_INTERLOCKED_EXCHANGE_ADD InterlockedExchangeAdd

[Akira Takahashi <faithandbrave@…>]

sorry, after code is follow:

#elif defined(_WIN32_WCE)

# define BOOST_INTERLOCKED_INCREMENT InterlockedIncrement
# define BOOST_INTERLOCKED_DECREMENT InterlockedDecrement
# define BOOST_INTERLOCKED_COMPARE_EXCHANGE InterlockedCompareExchange
# define BOOST_INTERLOCKED_EXCHANGE InterlockedExchange
# define BOOST_INTERLOCKED_EXCHANGE_ADD InterlockedExchangeAdd

[viboes]

Which boost library is including the detail/interlocked.hpp file?

Could you send the declarations of the InterlockedXXX functions you have on your WindowsCE platform and the version?

[faithandbrave@…]

detail/interlocked.hpp used by Boost.Thread. include place is boost/thread/win32/interlocked_read.hpp.

WindowsCE Version $(CEVER) is defined as 0x600.

_WIN32_WCE=$(CEVER)

InterlockedXXX function declaration is follow:

winbase.h

#ifdef __cplusplus
extern "C" {
#endif

...

#ifdef MIPS_R4000 /* or above */
...
#elif defined(_X86_)
...
#else // not _X86_

LONG
WINAPI
InterlockedIncrement(
    LONG volatile *lpAddend
    );


LONG
WINAPI
InterlockedDecrement(
    LONG volatile *lpAddend
    );


LONG
WINAPI
InterlockedExchange(
    LONG volatile *Target,
    LONG Value
    );

LONG
WINAPI
InterlockedCompareExchange(
    LONG volatile *Target,
    LONG Exchange,
    LONG Comperand
    );

LONG
WINAPI
InterlockedExchangeAdd(
    LONG volatile *lpAddend,
    LONG Value
    );

#define InterlockedTestExchange(Target, oldValue, newValue) \
    InterlockedCompareExchange((Target), (newValue), (oldValue))

#endif

...
#ifdef __cplusplus
}
#endif

[viboes]

Could changing the declarations to

extern "C" long __cdecl InterlockedIncrement( long volatile * );
extern "C" long __cdecl InterlockedDecrement( long volatile * );
extern "C" long __cdecl InterlockedCompareExchange( long volatile *, long, long );
extern "C" long __cdecl InterlockedExchange( long volatile *, long );
extern "C" long __cdecl InterlockedExchangeAdd( long volatile *, long );

solve also the issue?

[Akira Takahashi <faithandbrave@…>]

I think that change is OK. But unfortunately, I have already left from the Windows CE platform. I can't test this change.

[mboard182@…]

Replying to viboes:

Could changing the declarations to

extern "C" long __cdecl InterlockedIncrement( long volatile * );
extern "C" long __cdecl InterlockedDecrement( long volatile * );
extern "C" long __cdecl InterlockedCompareExchange( long volatile *, long, long );
extern "C" long __cdecl InterlockedExchange( long volatile *, long );
extern "C" long __cdecl InterlockedExchangeAdd( long volatile *, long );

solve also the issue?

Yes, this appears to solve the link error. Can you help me understand why??

AFAIK with 'extern "C"', the following have the same name (and indeed will not link for me on vc9.0/winCE6 arm)

void foo(int *i){...}
void foo(volatile int *i){...}

[viboes]

Committed in trunk revision 80042.

[viboes]

last update rolledback as it make regression test fail. Committed in trunk revision [80067]. I will try making the modification only when _WIN32_WCE==0x600.

[viboes]

Committed in trunk revision [80127].

[viboes]

Merged from trunk [80473].

[Martin.problemboost.Maurer@…]

I got the same error under Windows Vista, Visual Studio Express 2008, using 64 bit compiler and intrin.h my in own program. Same situation with boost 1.51.0 and 1.53.0 Could be because the above bugfix seems to be activated only for Windows CE, but i have Vista not Windows CE. Any idea?

[viboes]

Please, create a new ticket as even if your problem is related it is for a different platform. Could you tell me which macro characterize your platform? If you could provide a patch this would help a lot.

[anonymous]

Similar error under Vista, VS2008, 64 bit compiler: ​https://svn.boost.org/trac/boost/ticket/8485

[christian.vollmer@…]

I got the same problem with the same platform but for the current boost version of 1.55. However, commenting out the section that handled the special case for _WIN32_WCE >= 0x600 solved the issue. Now the code looks like this:

//#if _WIN32_WCE >= 0x600
//
//extern "C" long __cdecl _InterlockedIncrement( long volatile * );
//extern "C" long __cdecl _InterlockedDecrement( long volatile * );
//extern "C" long __cdecl _InterlockedCompareExchange( long volatile *, long, long );
//extern "C" long __cdecl _InterlockedExchange( long volatile *, long );
//extern "C" long __cdecl _InterlockedExchangeAdd( long volatile *, long );
//
//# define BOOST_INTERLOCKED_INCREMENT _InterlockedIncrement
//# define BOOST_INTERLOCKED_DECREMENT _InterlockedDecrement
//# define BOOST_INTERLOCKED_COMPARE_EXCHANGE _InterlockedCompareExchange
//# define BOOST_INTERLOCKED_EXCHANGE _InterlockedExchange
//# define BOOST_INTERLOCKED_EXCHANGE_ADD _InterlockedExchangeAdd
//
//#else
// under Windows CE we still have old-style Interlocked* functions

//extern "C" long __cdecl InterlockedIncrement( long* );
//extern "C" long __cdecl InterlockedDecrement( long* );
//extern "C" long __cdecl InterlockedCompareExchange( long*, long, long );
//extern "C" long __cdecl InterlockedExchange( long*, long );
//extern "C" long __cdecl InterlockedExchangeAdd( long*, long );

# define BOOST_INTERLOCKED_INCREMENT InterlockedIncrement
# define BOOST_INTERLOCKED_DECREMENT InterlockedDecrement
# define BOOST_INTERLOCKED_COMPARE_EXCHANGE InterlockedCompareExchange
# define BOOST_INTERLOCKED_EXCHANGE InterlockedExchange
# define BOOST_INTERLOCKED_EXCHANGE_ADD InterlockedExchangeAdd

//#endif

[viboes]

Peter please, could you take a look at this issue?

[Peter Dimov]

I don't have access to any Windows CE platform, so I'm not sure what I can contribute. I don't even understand what problem is being discussed here.

[James E. King, III]

PR out for this: ​https://github.com/boostorg/winapi/pull/58

[Andrey Semashev]

I've committed the fix in ​https://github.com/boostorg/winapi/commit/e5a59445bfe3f8f5f6077d1e7b4d5d91a02a10af. I'm not able to test the change and would appreciate if anyone here is able to try it.