> -----Original Message----- > From: Richard Earnshaw > Sent: Thursday, November 28, 2013 7:09 PM > To: Terry Guo > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [Patch, ARM] Add v7m specific extra rtx cost table > > On 28/11/13 10:34, Terry Guo wrote: > > Hello, > > > > This patch intends to add a specific extra rtx cost table for v7-m > > profile targets. Tested with gcc regression test, no new regressions. > > Is it OK to trunk? > > > > BR, > > Terry > > > > 2013-11-28 Terry Guo <terry....@arm.com> > > > > * config/arm/aarch-cost-tables.h (v7m_extra_costs): New table. > > > > You haven't mentioned the change in arm.c > > > > > m4-extra-cost-table-upstream-v1.txt > > > > > > diff --git a/gcc/config/arm/aarch-cost-tables.h > > b/gcc/config/arm/aarch-cost-tables.h > > index d3e7dd2..52e18a1 100644 > > --- a/gcc/config/arm/aarch-cost-tables.h > > +++ b/gcc/config/arm/aarch-cost-tables.h > > @@ -223,5 +223,105 @@ const struct cpu_cost_table > > cortexa53_extra_costs = }; > > > > > > +const struct cpu_cost_table v7m_extra_costs = > > As Kyrill says, this should be in arm.c > > OK with that change. > > R.
Thank you all. I will update the patch and commit it. BR, Terry
