Hi jasonwucj, > -----Original Message----- > From: Gcc-patches <gcc-patches- > bounces+kyrylo.tkachov=arm....@gcc.gnu.org> On Behalf Of Chung-Ju Wu > via Gcc-patches > Sent: Thursday, May 26, 2022 8:18 AM > To: Richard Earnshaw <richard.earns...@arm.com>; gcc-patches <gcc- > patc...@gcc.gnu.org> > Cc: jason...@anshingtek.com.tw > Subject: [PATCH 2/3][ARM] STAR-MC1 CPU Support - arm: Add individual > star-mc1 cost tables and cost functions > > Hi, > > Attached is the patch to provide star-mc1 specific cost functions and tables. > Given these individual implementation, developers are able to make > their own adjustment to fine-tune star-mc1 performance without affecting > other cpu configurations. > > Bootstrapped and tested on arm-none-eabi. > > Is it OK for trunk?
It looks like arm_star_mc1_tune, star_mc1_extra_costs and arm_star_mc1_branch_cost are identical to arm_v7m_tune, v7m_extra_costs and arm_cortex_m_branch_cost. I'd rather not duplicate those structures and functions in the master branch, as they provide a maintenance burden to the community. If some tuning parameters need to be modified in the future for better performance we can create star-mc1-specific structures on demand then. Thus, I think we don't want this patch. Thanks, Kyrill