On Thu, Jul 31, 2014 at 1:54 PM, Gopalasubramanian, Ganesh <ganesh.gopalasubraman...@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.