From 61da5e46b4db898a551a2fb5b489c5b72c3518fb Mon Sep 17 00:00:00 2001 From: Ramana Radhakrishnan Date: Thu, 5 Nov 2015 09:48:53 +0000 Subject: [PATCH] [Patch AArch64] Switch constant pools to separate rodata sections. Now that PR63304 is fixed and we have an option to address any part of the memory using adrp / add or adrp / ldr instructions it makes sense to switch out literal pools into their own mergeable sections by default. This would mean that by default we could now start getting the benefits of constant sharing across the board, potentially improving code size. The other advantage of doing so, for the security conscious is that this prevents intermingling of literal pools with code. Wilco's kindly done some performance measurements and suggests that there is not really a performance regression in doing this. I've looked at the code size for SPEC2k6 today at -Ofast and in general there is a good code size improvement as expected by sharing said constants. Tested on aarch64-none-elf with no regressions and bootstrapped and regression tested in my tree for a number of days now. 2015-11-05 Ramana Radhakrishnan * config/aarch64/aarch64.c (aarch64_can_use_per_function_literal_pools_p): New. (aarch64_use_blocks_for_constant_p): Adjust declaration and use aarch64_can_use_function_literal_pools_p. (aarch64_select_rtx_section): Update. From-SVN: r229795 --- gcc/ChangeLog | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 2e58c51ad70..60ad78a2170 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,11 @@ +2015-11-05 Ramana Radhakrishnan + + * config/aarch64/aarch64.c + (aarch64_can_use_per_function_literal_pools_p): New. + (aarch64_use_blocks_for_constant_p): Adjust declaration + and use aarch64_can_use_function_literal_pools_p. + (aarch64_select_rtx_section): Update. + 2015-11-05 Ilya Enkovich * targhooks.c (default_get_mask_mode): Use BLKmode in -- 2.30.2