This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, i386] Handle extended family cpuid info for AMD
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "Gopalasubramanian, Ganesh" <Ganesh dot Gopalasubramanian at amd dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 31 Jul 2014 14:37:43 +0200
- Subject: Re: [PATCH, i386] Handle extended family cpuid info for AMD
- Authentication-results: sourceware.org; auth=none
- References: <EB4625145972F94C9680D8CADD651615787D3293 at SATLEXDAG02 dot amd dot com> <CAFULd4bYmTe8H0DobUjsTV2ZkUjj3F66ofS-Fk4yfW08E=aK2g at mail dot gmail dot com> <EB4625145972F94C9680D8CADD651615787D32D3 at SATLEXDAG02 dot amd dot com> <CAFULd4aoB070f4rTZ9RsR4x4K89QZFaHmBKdMhmZQFf09M8DVQ at mail dot gmail dot com> <EB4625145972F94C9680D8CADD651615787D32E9 at SATLEXDAG02 dot amd dot com>
On Thu, Jul 31, 2014 at 1:54 PM, Gopalasubramanian, Ganesh
<Ganesh.Gopalasubramanian@amd.com> wrote:
>> Then just use:
>
>> + else if (has_avx2)
>> + processor = PROCESSOR_BDVER4;
>> else if (has_movbe)
>> processor = PROCESSOR_BTVER2;
>>- else if (has_avx2)
>>- processor = PROCESSOR_BDVER4;
>> else if (has_xsaveopt)
>
> In that case, with earlier GCC versions where we donât have bdver4 added, the fall back would be BTVER2,
> whereas a BD variant is more desirable.
I would like to have a check for a family at the beginning, something like:
if (name == signature_NSC_ebx)
processor = PROCESSOR_GEODE;
else if (family == 22)
{
if (has_movbe)
processor = PROCESSOR_BTVER2;
else
processor = PROCESSOR_BTVER1;
}
else if (has_avx2)
processor = PROCESSOR_BDVER4;
else if (has_xsaveopt)
Please also fix ChangeLog entry.
Uros.