Re: [PATCH][AARCH64] Set jump-align=4 for neoversen1

2020-01-20 Thread Richard Sandiford
Wilco Dijkstra writes: > Hi Kyrill & Richard, > >> I was leaving this to others in case it was obvious to them. On the >> basis that silence suggests it wasn't, :-) could you go into more details? >> Is it expected on first principles that jump alignment doesn't matter >> for Neoverse N1, or is t

Re: [PATCH][AARCH64] Set jump-align=4 for neoversen1

2020-01-17 Thread Wilco Dijkstra
Hi Kyrill & Richard, > I was leaving this to others in case it was obvious to them. On the > basis that silence suggests it wasn't, :-) could you go into more details? > Is it expected on first principles that jump alignment doesn't matter > for Neoverse N1, or is this purely based on experimenta

Re: [PATCH][AARCH64] Set jump-align=4 for neoversen1

2020-01-17 Thread Kyrill Tkachov
Hi Richard, Wilco, On 1/17/20 8:43 AM, Richard Sandiford wrote: Wilco Dijkstra writes: > Testing shows the setting of 32:16 for jump alignment has a significant codesize > cost, however it doesn't make a difference in performance. So set jump-align > to 4 to get 1.6% codesize improvement. I

Re: [PATCH][AARCH64] Set jump-align=4 for neoversen1

2020-01-17 Thread Richard Sandiford
Wilco Dijkstra writes: > Testing shows the setting of 32:16 for jump alignment has a significant > codesize > cost, however it doesn't make a difference in performance. So set jump-align > to 4 to get 1.6% codesize improvement. I was leaving this to others in case it was obvious to them. On the

Re: [PATCH][AARCH64] Set jump-align=4 for neoversen1

2020-01-16 Thread Wilco Dijkstra
ping Testing shows the setting of 32:16 for jump alignment has a significant codesize cost, however it doesn't make a difference in performance. So set jump-align to 4 to get 1.6% codesize improvement. OK for commit? ChangeLog 2019-12-24 Wilco Dijkstra * config/aarch64/aarch64.c (ne

[PATCH][AARCH64] Set jump-align=4 for neoversen1

2019-12-24 Thread Wilco Dijkstra
Testing shows the setting of 32:16 for jump alignment has a significant codesize cost, however it doesn't make a difference in performance. So set jump-align to 4 to get 1.6% codesize improvement. OK for commit? ChangeLog 2019-12-24 Wilco Dijkstra * config/aarch64/aarch64.c (neoverse