commit 89c785e5fe0a57483f62fdd84a8b5b4e365c1b95 Author: Kyrylo Tkachov Date: Thu May 7 12:07:51 2015 +0100 [AArch64][6/N] Implement TARGET_OPTION_SAVE/TARGET_OPTION_RESTORE diff --git a/gcc/config/aarch64/aarch64-protos.h b/gcc/config/aarch64/aarch64-protos.h index e4f5b00..fc1cec7 100644 --- a/gcc/config/aarch64/aarch64-protos.h +++ b/gcc/config/aarch64/aarch64-protos.h @@ -255,6 +255,7 @@ bool aarch64_gimple_fold_builtin (gimple_stmt_iterator *); bool aarch64_is_extend_from_extract (machine_mode, rtx, rtx); bool aarch64_is_long_call_p (rtx); bool aarch64_label_mentioned_p (rtx); +void aarch64_declare_function_name (FILE *, const char*, tree); bool aarch64_legitimate_pic_operand_p (rtx); bool aarch64_modes_tieable_p (machine_mode mode1, machine_mode mode2); diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index bb404ac..2891690 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -158,9 +158,6 @@ unsigned aarch64_architecture_version; /* The processor for which instructions should be scheduled. */ enum aarch64_processor aarch64_tune = cortexa53; -/* Mask to specify which instructions we are allowed to generate. */ -unsigned long aarch64_isa_flags = 0; - /* Mask to specify which instruction scheduling options should be used. */ unsigned long aarch64_tune_flags = 0; @@ -531,8 +528,8 @@ static const struct processor all_cores[] = }; -/* Target specification. These are populated as commandline arguments - are processed, or NULL if not specified. */ +/* Target specification. These are populated by the -march, -mtune, -mcpu + handling code or by target attributes. */ static const struct processor *selected_arch; static const struct processor *selected_cpu; static const struct processor *selected_tune; @@ -7563,7 +7560,7 @@ aarch64_override_options_internal (struct gcc_options *opts) it is usually called from within other parsing functions. */ char tmp_str[6]; strcpy (tmp_str, "+nofp"); - aarch64_parse_extension (tmp_str, &aarch64_isa_flags); + aarch64_parse_extension (tmp_str, &opts->x_aarch64_isa_flags); } initialize_aarch64_code_model (opts); @@ -7573,8 +7570,10 @@ aarch64_override_options_internal (struct gcc_options *opts) /* Validate a command-line -mcpu option. Parse the cpu and extensions (if any) specified in STR and throw errors if appropriate. Put the results if - they are valid in RES and ISA_FLAGS. */ -static void + they are valid in RES and ISA_FLAGS. Return whether the option is + valid. */ + +static bool aarch64_validate_mcpu (const char *str, const struct processor **res, unsigned long *isa_flags) { @@ -7582,7 +7581,7 @@ aarch64_validate_mcpu (const char *str, const struct processor **res, = aarch64_parse_cpu (str, res, isa_flags); if (parse_res == AARCH64_PARSE_OK) - return; + return true; switch (parse_res) { @@ -7598,12 +7597,16 @@ aarch64_validate_mcpu (const char *str, const struct processor **res, default: gcc_unreachable (); } + + return false; } /* Validate a command-line -march option. Parse the arch and extensions (if any) specified in STR and throw errors if appropriate. Put the - results, if they are valid, in RES and ISA_FLAGS. */ -static void + results, if they are valid, in RES and ISA_FLAGS. Return whether the + option is valid. */ + +static bool aarch64_validate_march (const char *str, const struct processor **res, unsigned long *isa_flags) { @@ -7611,7 +7614,7 @@ aarch64_validate_march (const char *str, const struct processor **res, = aarch64_parse_arch (str, res, isa_flags); if (parse_res == AARCH64_PARSE_OK) - return; + return true; switch (parse_res) { @@ -7627,19 +7630,23 @@ aarch64_validate_march (const char *str, const struct processor **res, default: gcc_unreachable (); } + + return false; } /* Validate a command-line -mtune option. Parse the cpu specified in STR and throw errors if appropriate. Put the - result, if it is valid, in RES. */ -static void + result, if it is valid, in RES. Return whether the option is + valid. */ + +static bool aarch64_validate_mtune (const char *str, const struct processor **res) { enum aarch64_parse_opt_result parse_res = aarch64_parse_tune (str, res); if (parse_res == AARCH64_PARSE_OK) - return; + return true; switch (parse_res) { @@ -7652,6 +7659,33 @@ aarch64_validate_mtune (const char *str, const struct processor **res) default: gcc_unreachable (); } + return false; +} + +/* Return the CPU corresponding to the enum CPU. + If it doesn't specify a cpu, return the default. */ + +static const struct processor * +aarch64_get_tune_cpu (enum aarch64_processor cpu) +{ + if (cpu != aarch64_none) + return &all_cores[cpu]; + + return &all_cores[TARGET_CPU_DEFAULT & 0x3f]; +} + +/* Return the architecture corresponding to the enum ARCH. + If it doesn't specify a valid architecture, return the default. */ + +static const struct processor * +aarch64_get_arch (enum aarch64_arch arch) +{ + if (arch != aarch64_no_arch) + return &all_architectures[arch]; + + const struct processor *cpu = &all_cores[TARGET_CPU_DEFAULT & 0x3f]; + + return &all_architectures[cpu->arch]; } /* Implement TARGET_OPTION_OVERRIDE. This is called once in the beginning @@ -7668,6 +7702,10 @@ aarch64_override_options (void) unsigned long arch_isa = 0; aarch64_isa_flags = 0; + bool valid_cpu = true; + bool valid_tune = true; + bool valid_arch = true; + selected_cpu = NULL; selected_arch = NULL; selected_tune = NULL; @@ -7676,13 +7714,15 @@ aarch64_override_options (void) If either of -march or -mtune is given, they override their respective component of -mcpu. */ if (aarch64_cpu_string) - aarch64_validate_mcpu (aarch64_cpu_string, &selected_cpu, &cpu_isa); + valid_cpu = aarch64_validate_mcpu (aarch64_cpu_string, &selected_cpu, + &cpu_isa); if (aarch64_arch_string) - aarch64_validate_march (aarch64_arch_string, &selected_arch, &arch_isa); + valid_arch = aarch64_validate_march (aarch64_arch_string, &selected_arch, + &arch_isa); if (aarch64_tune_string) - aarch64_validate_mtune (aarch64_tune_string, &selected_tune); + valid_tune = aarch64_validate_mtune (aarch64_tune_string, &selected_tune); /* If the user did not specify a processor, choose the default one for them. This will be the CPU set during configuration using @@ -7693,12 +7733,17 @@ aarch64_override_options (void) { selected_cpu = &all_cores[selected_arch->ident]; aarch64_isa_flags = arch_isa; + explicit_arch = selected_arch->arch; } else { - selected_cpu = &all_cores[TARGET_CPU_DEFAULT & 0x3f]; + /* Get default configure-time CPU. */ + selected_cpu = aarch64_get_tune_cpu (aarch64_none); aarch64_isa_flags = TARGET_CPU_DEFAULT >> 6; } + + if (selected_tune) + explicit_tune_core = selected_tune->ident; } /* If both -mcpu and -march are specified check that they are architecturally compatible, warn if they're not and prefer the -march ISA flags. */ @@ -7711,10 +7756,20 @@ aarch64_override_options (void) selected_arch->name); } aarch64_isa_flags = arch_isa; + explicit_arch = selected_arch->arch; + explicit_tune_core = selected_tune ? selected_tune->ident + : selected_cpu->ident; } /* -mcpu but no -march. */ else - aarch64_isa_flags = cpu_isa; + { + aarch64_isa_flags = cpu_isa; + explicit_tune_core = selected_tune ? selected_tune->ident + : selected_cpu->ident; + gcc_assert (selected_cpu); + selected_arch = &all_architectures[selected_cpu->arch]; + explicit_arch = selected_arch->arch; + } /* Set the arch as well as we will need it when outputing the .arch directive in assembly. */ @@ -7734,6 +7789,15 @@ aarch64_override_options (void) error ("Assembler does not support -mabi=ilp32"); #endif + /* Make sure we properly set up the explicit options. */ + if ((aarch64_cpu_string && valid_cpu) + || (aarch64_tune_string && valid_tune)) + gcc_assert (explicit_tune_core != aarch64_none); + + if ((aarch64_cpu_string && valid_cpu) + || (aarch64_arch_string && valid_arch)) + gcc_assert (explicit_arch != aarch64_no_arch); + aarch64_build_bitmask_table (); aarch64_override_options_internal (&global_options); @@ -7817,6 +7881,53 @@ initialize_aarch64_code_model (struct gcc_options *opts) aarch64_cmodel = opts->x_aarch64_cmodel_var; } +/* Print to F the architecture features specified by ISA_FLAGS. */ + +static void +aarch64_print_extension (FILE *f, unsigned long isa_flags) +{ + const struct aarch64_option_extension *opt = NULL; + + for (opt = all_extensions; opt->name != NULL; opt++) + if ((isa_flags & opt->flags_on) == opt->flags_on) + asm_fprintf (f, "+%s", opt->name); + + asm_fprintf (f, "\n"); +} + +/* TARGET_OPTION_SAVE is not needed at the moment. All the option saving is + already done correctly by the generic option saving code. */ + +/* Implements TARGET_OPTION_RESTORE. Restore the backend codegen decisions + using the information saved in PTR. */ + +static void +aarch64_option_restore (struct gcc_options *opts, struct cl_target_option *ptr) +{ + opts->x_explicit_tune_core = ptr->x_explicit_tune_core; + selected_tune = aarch64_get_tune_cpu (ptr->x_explicit_tune_core); + opts->x_explicit_arch = ptr->x_explicit_arch; + selected_arch = aarch64_get_arch (ptr->x_explicit_arch); + + aarch64_override_options_internal (opts); +} + +/* Implement TARGET_OPTION_PRINT. */ + +static void +aarch64_option_print (FILE *file, int indent, struct cl_target_option *ptr) +{ + const struct processor *cpu + = aarch64_get_tune_cpu (ptr->x_explicit_tune_core); + unsigned long isa_flags = ptr->x_aarch64_isa_flags; + const struct processor *arch = aarch64_get_arch (ptr->x_explicit_arch); + + fprintf (file, "%*sselected tune = %s\n", indent, "", cpu->name); + fprintf (file, "%*sselected arch = %s", indent, "", arch->name); + aarch64_print_extension (file, isa_flags); +} + + /* Return true if SYMBOL_REF X binds locally. */ static bool @@ -9824,6 +9935,42 @@ aarch64_asm_preferred_eh_data_format (int code ATTRIBUTE_UNUSED, int global) return (global ? DW_EH_PE_indirect : 0) | DW_EH_PE_pcrel | type; } +/* Implement ASM_DECLARE_FUNCTION_NAME. Output the ISA features used + by the function fndecl. */ + +void +aarch64_declare_function_name (FILE *stream, const char* name, + tree fndecl) +{ + tree target_parts = DECL_FUNCTION_SPECIFIC_TARGET (fndecl); + + struct cl_target_option *targ_options; + if (target_parts) + targ_options = TREE_TARGET_OPTION (target_parts); + else + targ_options = TREE_TARGET_OPTION (target_option_current_node); + gcc_assert (targ_options); + + const struct processor *this_arch + = aarch64_get_arch (targ_options->x_explicit_arch); + + asm_fprintf (asm_out_file, "\t.arch %s", this_arch->name); + aarch64_print_extension (asm_out_file, targ_options->x_aarch64_isa_flags); + + /* Print the cpu name we're tuning for in the comments, might be + useful to readers of the generated asm. */ + + const struct processor *this_tune + = aarch64_get_tune_cpu (targ_options->x_explicit_tune_core); + + asm_fprintf (asm_out_file, "\t" ASM_COMMENT_START ".tune %s\n", + this_tune->name); + + /* Don't forget the type directive for ELF. */ + ASM_OUTPUT_TYPE_DIRECTIVE (stream, name, "function"); + ASM_OUTPUT_LABEL (stream, name); +} + /* Emit load exclusive. */ static void @@ -10104,36 +10251,6 @@ aarch64_split_atomic_op (enum rtx_code code, rtx old_out, rtx new_out, rtx mem, aarch64_emit_post_barrier (model); } -static void -aarch64_print_extension (void) -{ - const struct aarch64_option_extension *opt = NULL; - - for (opt = all_extensions; opt->name != NULL; opt++) - if ((aarch64_isa_flags & opt->flags_on) == opt->flags_on) - asm_fprintf (asm_out_file, "+%s", opt->name); - - asm_fprintf (asm_out_file, "\n"); -} - -static void -aarch64_start_file (void) -{ - if (selected_arch) - { - asm_fprintf (asm_out_file, "\t.arch %s", selected_arch->name); - aarch64_print_extension (); - } - else if (selected_cpu) - { - const char *truncated_name - = aarch64_rewrite_selected_cpu (selected_cpu->name); - asm_fprintf (asm_out_file, "\t.cpu %s", truncated_name); - aarch64_print_extension (); - } - default_file_start(); -} - /* Target hook for c_mode_for_suffix. */ static machine_mode aarch64_c_mode_for_suffix (char suffix) @@ -12157,9 +12274,6 @@ aarch64_unspec_may_trap_p (const_rtx x, unsigned flags) #define TARGET_ASM_CAN_OUTPUT_MI_THUNK \ hook_bool_const_tree_hwi_hwi_const_tree_true -#undef TARGET_ASM_FILE_START -#define TARGET_ASM_FILE_START aarch64_start_file - #undef TARGET_ASM_OUTPUT_MI_THUNK #define TARGET_ASM_OUTPUT_MI_THUNK aarch64_output_mi_thunk @@ -12279,6 +12393,12 @@ aarch64_unspec_may_trap_p (const_rtx x, unsigned flags) #define TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE \ aarch64_override_options_after_change +#undef TARGET_OPTION_RESTORE +#define TARGET_OPTION_RESTORE aarch64_option_restore + +#undef TARGET_OPTION_PRINT +#define TARGET_OPTION_PRINT aarch64_option_print + #undef TARGET_PASS_BY_REFERENCE #define TARGET_PASS_BY_REFERENCE aarch64_pass_by_reference diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h index e91541a..6d792c4 100644 --- a/gcc/config/aarch64/aarch64.h +++ b/gcc/config/aarch64/aarch64.h @@ -221,7 +221,7 @@ extern unsigned aarch64_architecture_version; | AARCH64_FL_LOR | AARCH64_FL_RDMA) /* Macros to test ISA flags. */ -extern unsigned long aarch64_isa_flags; + #define AARCH64_ISA_CRC (aarch64_isa_flags & AARCH64_FL_CRC) #define AARCH64_ISA_CRYPTO (aarch64_isa_flags & AARCH64_FL_CRYPTO) #define AARCH64_ISA_FP (aarch64_isa_flags & AARCH64_FL_FP) @@ -439,6 +439,10 @@ extern unsigned long aarch64_isa_flags; #define ASM_PREFERRED_EH_DATA_FORMAT(CODE, GLOBAL) \ aarch64_asm_preferred_eh_data_format ((CODE), (GLOBAL)) +/* Output the assembly strings we want to add to a function definition. */ +#define ASM_DECLARE_FUNCTION_NAME(STR, NAME, DECL) \ + aarch64_declare_function_name (STR, NAME, DECL) + /* The register that holds the return address in exception handlers. */ #define AARCH64_EH_STACKADJ_REGNUM (R0_REGNUM + 4) #define EH_RETURN_STACKADJ_RTX gen_rtx_REG (Pmode, AARCH64_EH_STACKADJ_REGNUM) diff --git a/gcc/config/aarch64/aarch64.opt b/gcc/config/aarch64/aarch64.opt index e29d606..b81ea8a 100644 --- a/gcc/config/aarch64/aarch64.opt +++ b/gcc/config/aarch64/aarch64.opt @@ -21,6 +21,15 @@ HeaderInclude config/aarch64/aarch64-opts.h +TargetVariable +enum aarch64_processor explicit_tune_core = aarch64_none + +TargetVariable +enum aarch64_arch explicit_arch = aarch64_no_arch + +TargetVariable +unsigned long aarch64_isa_flags = 0 + ; The TLS dialect names to use with -mtls-dialect. Enum @@ -53,11 +62,11 @@ Target Report RejectNegative Mask(BIG_END) Assume target CPU is configured as big endian mgeneral-regs-only -Target Report RejectNegative Mask(GENERAL_REGS_ONLY) +Target Report RejectNegative Mask(GENERAL_REGS_ONLY) Save Generate code which uses only the general registers mfix-cortex-a53-835769 -Target Report Var(aarch64_fix_a53_err835769) Init(2) +Target Report Var(aarch64_fix_a53_err835769) Init(2) Save Workaround for ARM Cortex-A53 Erratum number 835769 mfix-cortex-a53-843419 @@ -69,19 +78,19 @@ Target Report RejectNegative InverseMask(BIG_END) Assume target CPU is configured as little endian mcmodel= -Target RejectNegative Joined Enum(cmodel) Var(aarch64_cmodel_var) Init(AARCH64_CMODEL_SMALL) +Target RejectNegative Joined Enum(cmodel) Var(aarch64_cmodel_var) Init(AARCH64_CMODEL_SMALL) Save Specify the code model mstrict-align -Target Report RejectNegative Mask(STRICT_ALIGN) +Target Report RejectNegative Mask(STRICT_ALIGN) Save Don't assume that unaligned accesses are handled by the system momit-leaf-frame-pointer -Target Report Save Var(flag_omit_leaf_frame_pointer) Init(2) +Target Report Var(flag_omit_leaf_frame_pointer) Init(2) Save Omit the frame pointer in leaf functions mtls-dialect= -Target RejectNegative Joined Enum(tls_type) Var(aarch64_tls_dialect) Init(TLS_DESCRIPTORS) +Target RejectNegative Joined Enum(tls_type) Var(aarch64_tls_dialect) Init(TLS_DESCRIPTORS) Save Specify TLS dialect march= @@ -101,7 +110,7 @@ Target RejectNegative Joined Enum(aarch64_abi) Var(aarch64_abi) Init(AARCH64_ABI -mabi=ABI Generate code that conforms to the specified ABI moverride= -Target RejectNegative ToLower Joined Var(aarch64_override_tune_string) +Target RejectNegative ToLower Joined Var(aarch64_override_tune_string) Save -moverride=STRING Power users only! Override CPU optimization parameters Enum