On Wed, Dec 18, 2013 at 4:38 PM, Gopalasubramanian, Ganesh <ganesh.gopalasubraman...@amd.com> wrote: > > Ping! > > "Gopalasubramanian, Ganesh" <ganesh.gopalasubraman...@amd.com> wrote: > > >> Yes, I figured that was the original idea behind it, but the final family of >> the jaguar processors seems to have become 16h instead of 14h (bobcat) at >> some point. > > Yes. It is amdfam16h. I was supposed to pass on some comments on the patch. > 1. Amdfam16h for Jaguar. > 2. For Jaguar, the priority needs to be AVX (AVX got included into the Jaguar > ISA). > > I have a doubt! What would be done if priority is set to "F_FMA4" instead of > "F_XOP" for bdver1?
XOP enables FMA4, so it should be better to set priority to P_PROC_XOP. From config/i386/i386-common.c: #define OPTION_MASK_ISA_XOP_SET \ (OPTION_MASK_ISA_XOP | OPTION_MASK_ISA_FMA4_SET) Looking at processor_dispatch_table, bdver1 doesn't support F_FMA, so the proposed patch fixes an error here. Uros.
