This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: PATCH: Support multiarch for i686
- From: "H.J. Lu" <hjl dot tools at gmail dot com>
- To: Ulrich Drepper <drepper at redhat dot com>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Fri, 31 Jul 2009 06:46:29 -0700
- Subject: Re: PATCH: Support multiarch for i686
- References: <20090730213311.GA15372@lucon.org> <4A727B8A.1040708@redhat.com>
On Thu, Jul 30, 2009 at 10:05 PM, Ulrich Drepper<drepper@redhat.com> wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> H.J. Lu wrote:
>
>> +#ifdef __x86_64__
>
> Don't do that. ?Define a macro in the i386 version saying this info
> isn't wanted and then test it here. ?Correct all the places where you
> use this type of #ifdef.
I will do it.
>
>> @@ -530,9 +522,7 @@ init_cacheinfo (void)
>> ? ? ? ? /* Query until desired cache level is enumerated. ?*/
>> ? ? ? ? do
>> ? ? ? ? ? {
>> - ? ? ? ? ? ? ?asm volatile ("cpuid"
>> - ? ? ? ? ? ? ? ? ? ? ? ? : "=a" (eax), "=b" (ebx), "=c" (ecx), "=d" (edx)
>> - ? ? ? ? ? ? ? ? ? ? ? ? : "0" (4), "2" (i++));
>> + ? ? ? ? ? __cpuid_count (4, i++, eax, ebx, ecx, edx);
>
> __cpuid_count is a bit too recent. ?Not even gcc 4.3 has it.
>
Should it check __cpuid_count or provide __cpuid_count if not available?
Thanks.
--
H.J.