From: Junio C Hamano <gitster@pobox.com>
Date: Tue, 01 May 2012 17:33:30 GMT
Subject: Re: [PATCH] Choose XDL_FAST_HASH code on sizeof(long) instead of __WORDSIZE
Message-ID: <7vhavzajrp.fsf@alter.siamese.dyndns.org>
In-Reply-To: <399c9a1f0dd548da3144eff3c50d04a8f343ab24.1335860033.git.trast@student.ethz.ch>

Thomas Rast <trast@student.ethz.ch> writes:

> Darwin does not define __WORDSIZE, and compiles the 32-bit code path
> on 64-bit systems, resulting in a totally broken git.
>
> I could not find an alternative -- other than the platform symbols
> (__x86_64__ etc.) -- that does the test in the preprocessor.  However,
> we can also just test for the size of a 'long', which is what really
> matters here.  Any compiler worth its salt will leave only the branch
> relevant for its platform, and indeed on Linux/GCC the numbers don't
> change:

Thanks.