[m-rev.] for review: Account for Winpthreads in configure.
Julien Fischer
jfischer at opturion.com
Wed Aug 13 15:56:30 AEST 2014
On Wed, 13 Aug 2014, Peter Wang wrote:
> On Wed, 13 Aug 2014 15:01:00 +1000 (EST), Julien Fischer <jfischer at opturion.com> wrote:
>>
>> On Wed, 13 Aug 2014, Peter Wang wrote:
>>
>>> Branches: master, 14.01
>>>
>>> configure.ac:
>>> Always build Boehm GC with GC_WIN32_PTHREADS when pthread.h is
>>> detected on MinGW* systems.
>>
>> Even in non par grades when parallel marking is not enabled? What
>> pthreads calls could we make then then? I think libc on MinGW-w64
>> requires winpthread to support some C11 features, so maybe it's always
>> needed on that system, but shouldn't it remain possible to build non
>> .par grades on plain MinGW without requiring pthreads?
>
> I guess it wasn't clear that it only affects .par grades and the par_gc
> library. New description:
>
> configure.ac:
> Always build the par_gc library with GC_WIN32_PTHREADS when
> pthread.h is detected on MinGW* systems. Previously, if the
> pthread implementation is not pthread-win32 we would fallback to
> building Boehm GC with GC_WIN32_THREADS (no "P"), so it would
> NOT provide pthread wrapper functions. But since pthread.h was
> found the Mercury implementation would make pthread calls, calls
> which are not intercepted by Boehm GC, and thus unsafe.
Yes, that's more clear -- the rest of the change is fine.
Cheers,
Julien.
More information about the reviews
mailing list