On Thu, Dec 3, 2020 at 11:13 PM Jeff Law via Gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > > > On 12/3/20 8:29 AM, Kumar, Venkataramanan via Gcc-patches wrote: > > [AMD Public Use] > > > > > > Hi Maintainers, > > > > PFA, the patch that enables support for the next generation AMD Zen3 CPU > > via -march=znver3. > > This is a very basic enablement patch. As of now the cost, tuning and > > scheduler changes are kept same as znver2. > > Further changes to the cost and tunings will be done later. > > > > Ok for trunk ? > > > > Regards, > > Venkat. > > > > > > X86_64-Enable-support-for-next-generation-AMD-Znver3.patch > > > > From ef7bd7d02e98d86ff32fa0dad6bc1d0802bd32aa Mon Sep 17 00:00:00 2001 > > From: Venkataramanan Kumar <venkataramanan.ku...@amd.com> > > Date: Thu, 3 Dec 2020 17:32:53 +0530 > > Subject: [PATCH] X86_64: Enable support for next generation AMD Zen3 CPU. > > > > 2020-12-03 Venkataramanan Kumar <venkataramanan.ku...@amd.com> > > Sharavan Kumar <shravan.ku...@amd.com> > > > > gcc/ChangeLog: > > > > * common/config/i386/cpuinfo.h (get_amd_cpu) recognize znver3. > > * common/config/i386/i386-common.c (processor_names): Add > > znver3. > > (processor_alias_table): Add znver3 and AMDFAM19H entry. > > * common/config/i386/i386-cpuinfo.h (processor_types): Add > > AMDFAM19H. > > (processor_subtypes): AMDFAM19H_ZNVER3. > > * config.gcc (i[34567]86-*-linux* | ...): Likewise. > > * config/i386/driver-i386.c: (host_detect_local_cpu): Let > > -march=native recognize znver3 processors. > > * config/i386/i386-c.c (ix86_target_macros_internal): Add > > znver3. > > * config/i386/i386-options.c (m_znver3): New definition. > > (m_ZNVER): Include m_znver3. > > (processor_cost_table): Add znver3. > > * config/i386/i386.c (ix86_reassociation_width): Likewise. > > * config/i386/i386.h (TARGET_znver3): New definition. > > (enum processor_type): Add PROCESSOR_ZNVER3. > > * config/i386/i386.md (define_attr "cpu"): Add znver3. > > * config/i386/x86-tune-sched.c: (ix86_issue_rate): Likewise. > > (ix86_adjust_cost): Likewise. > > * config/i386/x86-tune.def (X86_TUNE_AVOID_256FMA_CHAINS: > > Likewise. > > * config/i386/znver1.md: Add new reservations for znver3. > > * doc/extend.texi: Add details about znver3. > > * doc/invoke.texi: Likewise. > Normally I would consider this inappropriate for stage3, but AFAICT the > risk profile of this patch should be small. Ultimately it's up to Uros > and I'll support whatever decision he makes.
Per rule changes to targets are allowed at any point per discretion of target maintainers. Heck, we even accept _new_ targets during stage3/4! So it's clearly appropriate at this stage but of course target maintainers need to ack changes in their area. Richard. > > Jeff >