diff mbox series

RISC-V: override alignment of function/jump/loop

Message ID CAHtqR7Wfr5n_mY9YJT4yAzRxh7Jna5CAhr5LRCUtcxwXgyLMEQ@mail.gmail.com
State New
Headers show
Series RISC-V: override alignment of function/jump/loop | expand

Commit Message

Wang Pengcheng Oct. 17, 2024, 12:41 p.m. UTC 
Just like what AArch64 has done.

Signed-off-by: Wang Pengcheng

gcc/ChangeLog:

* config/riscv/riscv.cc (struct riscv_tune_param): Add new
tune options.
(riscv_override_options_internal): Override the default alignment
when not optimizing for size.
---
gcc/config/riscv/riscv.cc | 15 +++++++++++++++
1 file changed, 15 insertions(+)

the performance cost is too great, even for -Os. Similarly, if
diff mbox series

Patch

diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc
index 3ac40234345..7d6fc1429b5 100644
--- a/gcc/config/riscv/riscv.cc
+++ b/gcc/config/riscv/riscv.cc
@@ -295,6 +295,9 @@  struct riscv_tune_param
bool overlap_op_by_pieces;
unsigned int fusible_ops;
const struct cpu_vector_cost *vec_costs;
+ const char *function_align = nullptr;
+ const char *jump_align = nullptr;
+ const char *loop_align = nullptr;
};


@@ -10283,6 +10286,18 @@  riscv_override_options_internal (struct
gcc_options *opts)
? &optimize_size_tune_info
: cpu->tune_param;

+ /* If not optimizing for size, set the default
+ alignment to what the target wants. */
+ if (!opts->x_optimize_size)
+ {
+ if (opts->x_flag_align_loops && !opts->x_str_align_loops)
+ opts->x_str_align_loops = tune_param->loop_align;
+ if (opts->x_flag_align_jumps && !opts->x_str_align_jumps)
+ opts->x_str_align_jumps = tune_param->jump_align;
+ if (opts->x_flag_align_functions && !opts->x_str_align_functions)
+ opts->x_str_align_functions = tune_param->function_align;
+ }
+
/* Use -mtune's setting for slow_unaligned_access, even when optimizing
for size. For architectures that trap and emulate unaligned accesses,