Jagasia, Harsha - RE: [patch] tuning gcc for AMDFAM10 processor (patch 1) (original) (raw)

This is the mail archive of the gcc-patches@gcc.gnu.orgmailing list for the GCC project.

• From: "Jagasia, Harsha"
• To: "Jakub Jelinek"
• Cc: "Richard Henderson" , gcc-patches at gcc dot gnu dot org, "Meissner, Michael"
• Date: Mon, 5 Feb 2007 10:55:48 -0600
• Subject: RE: [patch] tuning gcc for AMDFAM10 processor (patch 1)

Fixed in the attached.

Regards, Harsha

Harsha Jagasia GNU Tools Team (SEE) 512-602-1775 Advanced Micro Devices harsha.jagasia@amd.com Austin, TX

-----Original Message----- From: Jakub Jelinek [mailto:jakub@redhat.com] Sent: Saturday, February 03, 2007 3:47 AM To: Jagasia, Harsha Cc: Richard Henderson; gcc-patches@gcc.gnu.org Subject: Re: [patch] tuning gcc for AMDFAM10 processor (patch 1)

On Fri, Feb 02, 2007 at 05:56:35PM -0600, Jagasia, Harsha wrote: @@ -1738,6 +1749,10 @@ override_options (void) | PTA_3DNOW_A | PTA_SSE | PTA_SSE2}, {"generic32", PROCESSOR_GENERIC32, 0 /* flags are only used for

march switch. / }, {"generic64", PROCESSOR_GENERIC64, PTA_64BIT / flags are only used for -march switch. */ },

•  {"amdfam10", PROCESSOR_K8, PTA_MMX | PTA_PREFETCH_SSE |

PTA_3DNOW

•                             | PTA_64BIT | PTA_3DNOW_A | PTA_SSE |

PTA_SSE2

•                             | PTA_SSE3 | PTA_POPCNT | PTA_ABM |

PTA_SSE4A

•                             | PTA_CX16},

  };

  int const pta_size = ARRAY_SIZE (processor_alias_table);

Just a minor note, could you keep generic32/generic64 last in the table? They are somewhat special, so it is better not to burry them somewhere in the middle.

Jakub

Attachment:gcc-amdfam10-1.patch
Description: gcc-amdfam10-1.patch