On Wed, Jun 22, 2016 at 09:12:25AM +0100, Kyrill Tkachov wrote: > Hi James, > > On 21/06/16 17:38, James Greenhalgh wrote: > >On Tue, Jun 21, 2016 at 04:55:43PM +0100, Kyrill Tkachov wrote: > >>Hi all, > >> > >>This is a rebase of https://gcc.gnu.org/ml/gcc-patches/2016-06/msg00403.html > >>on top of Evandro's changes. > >>Also, to elaborate on the original posting, the initial tuning structure is > >>based on the Cortex-A57 one but with the issue rate set to 2, FMA steering > >>turned off and ADRP+LDR fusion enabled. > >I see you've also chosen to use the generic_branch_cost costs for > >branches. As you didn't mention it explicitly here, was that intentional? > > > > Ah, that was copied from the Cortex-a72 tuning. I didn't spend any time > experimenting with it. generic_branch_costs should be good enough for the > initial enablement. I can change it to cortexa57_branch_cost if you'd like. > Or we can do it separately later (I suspect Cortex-A72 should use those costs > too.)
Yes, I'm more than happy for it to be a follow-up. We'll probably need to revisit the settings for a few of the cores once the if-convert cost model changes I've been working on go in anyway. Thanks again, James
