From e9e0aa2c9677074a7912521f4fa1aebf53640198 Mon Sep 17 00:00:00 2001 From: Diego Novillo Date: Wed, 11 Apr 2007 16:14:06 +0000 Subject: [PATCH] re PR tree-optimization/30735 (50% slow down due to mem-ssa merge) PR 30735 PR 31090 * doc/invoke.texi: Document --params max-aliased-vops and avg-aliased-vops. * tree-ssa-operands.h (get_mpt_for, dump_memory_partitions, debug_memory_partitions): Move to tree-flow.h * params.h (AVG_ALIASED_VOPS): Define. * tree-ssa-alias.c (struct mp_info_def): Remove. Update all users. (mp_info_t): Likewise. (get_mem_sym_stats_for): New. (set_memory_partition): Move from tree-flow-inline.h. (mark_non_addressable): Only clear the set of symbols for the partition if it exists. (dump_memory_partitions): Move from tree-ssa-operands.c (debug_memory_partitions): Likewise. (need_to_partition_p): New. (dump_mem_ref_stats): New. (debug_mem_ref_stats): New. (dump_mem_sym_stats): New. (debug_mem_sym_stats): New. (update_mem_sym_stats_from_stmt): New. (compare_mp_info_entries): New. (mp_info_cmp): Call it. (sort_mp_info): Change argument to a list of mem_sym_stats_t objects. (get_mpt_for): Move from tree-ssa-operands.c. (find_partition_for): New. (create_partition_for): Remove. (estimate_vop_reduction): New. (update_reference_counts): New. (build_mp_info): New. (compute_memory_partitions): Refactor. Document new heuristic. Call build_mp_info, update_reference_counts, find_partition_for and estimate_vop_reduction. (compute_may_aliases): Populate virtual operands before calling debugging dumps. (delete_mem_sym_stats): New. (delete_mem_ref_stats): New. (init_mem_ref_stats): New. (init_alias_info): Call it. (maybe_create_global_var): Remove alias_info argument. Get number of call sites and number of pure/const call sites from gimple_mem_ref_stats(). (dump_alias_info): Call dump_memory_partitions first. (dump_points_to_info_for): Show how many times a pointer has been dereferenced. * opts.c (decode_options): For -O2 set --param max-aliased-vops to 500. For -O3 set --param max-aliased-vops to 1000 and --param avg-aliased-vops to 3. * fortran/options.c (gfc_init_options): Remove assignment to MAX_ALIASED_VOPS. * tree-flow-inline.h (gimple_mem_ref_stats): New. * tree-dfa.c (dump_variable): Dump memory reference statistics. Dump NO_ALIAS* settings. (referenced_var_lookup): Tidy. (mem_sym_stats): New. * tree-ssa-copy.c (may_propagate_copy): Return true if DEST and ORIG are different SSA names for a memory partition. * tree-ssa.c (delete_tree_ssa): Call delete_mem_ref_stats. * tree-flow.h (struct mem_sym_stats_d): Define. (mem_sym_stats_t): Define. (struct mem_ref_stats_d): Define. (struct gimple_df): Add field mem_ref_stats. (enum noalias_state): Define. (struct var_ann_d): Add bitfield noalias_state. (mem_sym_stats, delete_mem_ref_stats, dump_mem_ref_stats, debug_mem_ref_stats, debug_memory_partitions, debug_mem_sym_stats): Declare. * tree-ssa-structalias.c (update_alias_info): Update call sites, pure/const call sites and asm sites in structure returned by gimple_mem_ref_stats. Remove local variable IS_POTENTIAL_DEREF. Increase NUM_DEREFS if the memory expression is a potential dereference. Call update_mem_sym_stats_from_stmt. If the memory references memory, call update_mem_sym_stats_from_stmt for all the direct memory symbol references found. (intra_create_variable_infos): Set noalias_state field for pointer arguments according to the value of flag_argument_noalias. * tree-ssa-structalias.h (struct alias_info): Remove fields num_calls_found and num_pure_const_calls_found. (update_mem_sym_stats_from_stmt): Declare. * params.def (PARAM_MAX_ALIASED_VOPS): Change description. Set default value to 100. (PARAM_AVG_ALIASED_VOPS): Define. From-SVN: r123719 --- gcc/ChangeLog | 94 +++ gcc/doc/invoke.texi | 27 +- gcc/fortran/options.c | 4 - gcc/opts.c | 7 + gcc/params.def | 9 +- gcc/params.h | 2 + gcc/tree-dfa.c | 67 +- gcc/tree-flow-inline.h | 32 +- gcc/tree-flow.h | 125 +++- gcc/tree-ssa-alias.c | 1285 ++++++++++++++++++++++++++++-------- gcc/tree-ssa-copy.c | 12 +- gcc/tree-ssa-operands.c | 86 --- gcc/tree-ssa-operands.h | 4 - gcc/tree-ssa-structalias.c | 117 +++- gcc/tree-ssa-structalias.h | 7 +- gcc/tree-ssa.c | 47 ++ gcc/tree-vrp.c | 7 +- 17 files changed, 1467 insertions(+), 465 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 7d85c96bd31..87ce15410e4 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,97 @@ +2007-04-11 Diego Novillo + + PR 30735 + PR 31090 + * doc/invoke.texi: Document --params max-aliased-vops and + avg-aliased-vops. + * tree-ssa-operands.h (get_mpt_for, dump_memory_partitions, + debug_memory_partitions): Move to tree-flow.h + * params.h (AVG_ALIASED_VOPS): Define. + * tree-ssa-alias.c (struct mp_info_def): Remove. Update all + users. + (mp_info_t): Likewise. + (get_mem_sym_stats_for): New. + (set_memory_partition): Move from tree-flow-inline.h. + (mark_non_addressable): Only clear the set of symbols for the + partition if it exists. + (dump_memory_partitions): Move from tree-ssa-operands.c + (debug_memory_partitions): Likewise. + (need_to_partition_p): New. + (dump_mem_ref_stats): New. + (debug_mem_ref_stats): New. + (dump_mem_sym_stats): New. + (debug_mem_sym_stats): New. + (update_mem_sym_stats_from_stmt): New. + (compare_mp_info_entries): New. + (mp_info_cmp): Call it. + (sort_mp_info): Change argument to a list of mem_sym_stats_t + objects. + (get_mpt_for): Move from tree-ssa-operands.c. + (find_partition_for): New. + (create_partition_for): Remove. + (estimate_vop_reduction): New. + (update_reference_counts): New. + (build_mp_info): New. + (compute_memory_partitions): Refactor. + Document new heuristic. + Call build_mp_info, update_reference_counts, + find_partition_for and estimate_vop_reduction. + (compute_may_aliases): Populate virtual operands before + calling debugging dumps. + (delete_mem_sym_stats): New. + (delete_mem_ref_stats): New. + (init_mem_ref_stats): New. + (init_alias_info): Call it. + (maybe_create_global_var): Remove alias_info argument. + Get number of call sites and number of pure/const call sites + from gimple_mem_ref_stats(). + (dump_alias_info): Call dump_memory_partitions first. + (dump_points_to_info_for): Show how many times a pointer has + been dereferenced. + * opts.c (decode_options): For -O2 set --param + max-aliased-vops to 500. + For -O3 set --param max-aliased-vops to 1000 and --param + avg-aliased-vops to 3. + * fortran/options.c (gfc_init_options): Remove assignment to + MAX_ALIASED_VOPS. + * tree-flow-inline.h (gimple_mem_ref_stats): New. + * tree-dfa.c (dump_variable): Dump memory reference + statistics. + Dump NO_ALIAS* settings. + (referenced_var_lookup): Tidy. + (mem_sym_stats): New. + * tree-ssa-copy.c (may_propagate_copy): Return true if DEST + and ORIG are different SSA names for a memory partition. + * tree-ssa.c (delete_tree_ssa): Call delete_mem_ref_stats. + * tree-flow.h (struct mem_sym_stats_d): Define. + (mem_sym_stats_t): Define. + (struct mem_ref_stats_d): Define. + (struct gimple_df): Add field mem_ref_stats. + (enum noalias_state): Define. + (struct var_ann_d): Add bitfield noalias_state. + (mem_sym_stats, delete_mem_ref_stats, dump_mem_ref_stats, + debug_mem_ref_stats, debug_memory_partitions, + debug_mem_sym_stats): Declare. + * tree-ssa-structalias.c (update_alias_info): Update call + sites, pure/const call sites and asm sites in structure + returned by gimple_mem_ref_stats. + Remove local variable IS_POTENTIAL_DEREF. + Increase NUM_DEREFS if the memory expression is a potential + dereference. + Call update_mem_sym_stats_from_stmt. + If the memory references memory, call + update_mem_sym_stats_from_stmt for all the direct memory + symbol references found. + (intra_create_variable_infos): Set noalias_state field for + pointer arguments according to the value of + flag_argument_noalias. + * tree-ssa-structalias.h (struct alias_info): Remove fields + num_calls_found and num_pure_const_calls_found. + (update_mem_sym_stats_from_stmt): Declare. + * params.def (PARAM_MAX_ALIASED_VOPS): Change description. + Set default value to 100. + (PARAM_AVG_ALIASED_VOPS): Define. + 2007-04-11 Richard Guenther PR middle-end/31530 diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 281c634040b..5b104ccbbc4 100644 --- a/gcc/doc/invoke.texi +++ b/gcc/doc/invoke.texi @@ -6588,10 +6588,29 @@ The maximum instructions CSE process before flushing. The default is 1000. @item max-aliased-vops -Maximum number of virtual operands per statement allowed to represent -aliases before triggering the alias grouping heuristic. Alias -grouping reduces compile times and memory consumption needed for -aliasing at the expense of precision loss in alias information. +Maximum number of virtual operands per function allowed to represent +aliases before triggering the alias partitioning heuristic. Alias +partitioning reduces compile times and memory consumption needed for +aliasing at the expense of precision loss in alias information. The +default value for this parameter is 100 for -O1, 500 for -O2 and 1000 +for -O3. + +Notice that if a function contains more memory statements than the +value of this parameter, it is not really possible to achieve this +reduction. In this case, the compiler will use the number of memory +statements as the value for @option{max-aliased-vops}. + +@item avg-aliased-vops + +Average number of virtual operands per statement allowed to represent +aliases before triggering the alias partitioning heuristic. This +works in conjunction with @option{max-aliased-vops}. If a function +contains more than @option{max-aliased-vops} virtual operators, then +memory symbols will be grouped into memory partitions until either the +total number of virtual operators is below @option{max-aliased-vops} +or the average number of virtual operators per memory statement is +below @option{avg-aliased-vops}. The default value for this parameter +is 1 for -O1 and -O2, and 3 for -O3. @item ggc-min-expand diff --git a/gcc/fortran/options.c b/gcc/fortran/options.c index 96bedabffb1..31c40768624 100644 --- a/gcc/fortran/options.c +++ b/gcc/fortran/options.c @@ -117,10 +117,6 @@ gfc_init_options (unsigned int argc ATTRIBUTE_UNUSED, /* -fshort-enums can be default on some targets. */ gfc_option.fshort_enums = targetm.default_short_enums (); - /* Increase MAX_ALIASED_VOPS to account for different characteristics - of Fortran regarding VOPs. */ - MAX_ALIASED_VOPS = 50; - return CL_Fortran; } diff --git a/gcc/opts.c b/gcc/opts.c index 4878b07c7c2..195e4e18c3c 100644 --- a/gcc/opts.c +++ b/gcc/opts.c @@ -506,6 +506,9 @@ decode_options (unsigned int argc, const char **argv) /* PRE tends to generate bigger code. */ flag_tree_pre = 1; } + + /* Allow more virtual operators to increase alias precision. */ + set_param_value ("max-aliased-vops", 500); } if (optimize >= 3) @@ -513,6 +516,10 @@ decode_options (unsigned int argc, const char **argv) flag_inline_functions = 1; flag_unswitch_loops = 1; flag_gcse_after_reload = 1; + + /* Allow even more virtual operators. */ + set_param_value ("max-aliased-vops", 1000); + set_param_value ("avg-aliased-vops", 3); } if (optimize < 2 || optimize_size) diff --git a/gcc/params.def b/gcc/params.def index 3725148c99e..72d51f8e75b 100644 --- a/gcc/params.def +++ b/gcc/params.def @@ -530,8 +530,13 @@ DEFPARAM(PARAM_MAX_RELOAD_SEARCH_INSNS, DEFPARAM(PARAM_MAX_ALIASED_VOPS, "max-aliased-vops", - "The maximum number of virtual operators per statement allowed to represent aliases before triggering alias grouping", - 10, 0, 0) + "The maximum number of virtual operators that a function is allowed to have before triggering memory partitioning heuristics", + 100, 0, 0) + +DEFPARAM(PARAM_AVG_ALIASED_VOPS, + "avg-aliased-vops", + "The average number of virtual operators that memory statements are allowed to have before triggering memory partitioning heuristics", + 1, 0, 0) DEFPARAM(PARAM_MAX_SCHED_REGION_BLOCKS, "max-sched-region-blocks", diff --git a/gcc/params.h b/gcc/params.h index 296db6141f5..386e781c26a 100644 --- a/gcc/params.h +++ b/gcc/params.h @@ -146,6 +146,8 @@ typedef enum compiler_param PARAM_VALUE (PARAM_SMS_LOOP_AVERAGE_COUNT_THRESHOLD) #define MAX_ALIASED_VOPS \ PARAM_VALUE (PARAM_MAX_ALIASED_VOPS) +#define AVG_ALIASED_VOPS \ + PARAM_VALUE (PARAM_AVG_ALIASED_VOPS) #define INTEGER_SHARE_LIMIT \ PARAM_VALUE (PARAM_INTEGER_SHARE_LIMIT) #define MAX_LAST_VALUE_RTL \ diff --git a/gcc/tree-dfa.c b/gcc/tree-dfa.c index 70563e70646..bc070233198 100644 --- a/gcc/tree-dfa.c +++ b/gcc/tree-dfa.c @@ -346,34 +346,54 @@ dump_variable (FILE *file, tree var) if (TREE_THIS_VOLATILE (var)) fprintf (file, ", is volatile"); + if (mem_sym_stats (cfun, var)) + { + mem_sym_stats_t stats = mem_sym_stats (cfun, var); + fprintf (file, ", direct reads: %ld", stats->num_direct_reads); + fprintf (file, ", direct writes: %ld", stats->num_direct_writes); + fprintf (file, ", indirect reads: %ld", stats->num_indirect_reads); + fprintf (file, ", indirect writes: %ld", stats->num_indirect_writes); + fprintf (file, ", read frequency: %ld", stats->frequency_reads); + fprintf (file, ", write frequency: %ld", stats->frequency_writes); + } + if (is_call_clobbered (var)) { + const char *s = ""; var_ann_t va = var_ann (var); unsigned int escape_mask = va->escape_mask; fprintf (file, ", call clobbered"); fprintf (file, " ("); if (escape_mask & ESCAPE_STORED_IN_GLOBAL) - fprintf (file, ", stored in global"); + { fprintf (file, "%sstored in global", s); s = ", "; } if (escape_mask & ESCAPE_TO_ASM) - fprintf (file, ", goes through ASM"); + { fprintf (file, "%sgoes through ASM", s); s = ", "; } if (escape_mask & ESCAPE_TO_CALL) - fprintf (file, ", passed to call"); + { fprintf (file, "%spassed to call", s); s = ", "; } if (escape_mask & ESCAPE_BAD_CAST) - fprintf (file, ", bad cast"); + { fprintf (file, "%sbad cast", s); s = ", "; } if (escape_mask & ESCAPE_TO_RETURN) - fprintf (file, ", returned from func"); + { fprintf (file, "%sreturned from func", s); s = ", "; } if (escape_mask & ESCAPE_TO_PURE_CONST) - fprintf (file, ", passed to pure/const"); + { fprintf (file, "%spassed to pure/const", s); s = ", "; } if (escape_mask & ESCAPE_IS_GLOBAL) - fprintf (file, ", is global var"); + { fprintf (file, "%sis global var", s); s = ", "; } if (escape_mask & ESCAPE_IS_PARM) - fprintf (file, ", is incoming pointer"); + { fprintf (file, "%sis incoming pointer", s); s = ", "; } if (escape_mask & ESCAPE_UNKNOWN) - fprintf (file, ", unknown escape"); - fprintf (file, " )"); + { fprintf (file, "%sunknown escape", s); s = ", "; } + fprintf (file, ")"); } + if (ann->noalias_state == NO_ALIAS) + fprintf (file, ", NO_ALIAS (does not alias other NO_ALIAS symbols)"); + else if (ann->noalias_state == NO_ALIAS_GLOBAL) + fprintf (file, ", NO_ALIAS_GLOBAL (does not alias other NO_ALIAS symbols" + " and global vars)"); + else if (ann->noalias_state == NO_ALIAS_ANYTHING) + fprintf (file, ", NO_ALIAS_ANYTHING (does not alias any other symbols)"); + if (gimple_default_def (cfun, var)) { fprintf (file, ", default def: "); @@ -618,8 +638,8 @@ referenced_var_lookup (unsigned int uid) { struct int_tree_map *h, in; in.uid = uid; - h = (struct int_tree_map *) htab_find_with_hash (gimple_referenced_vars (cfun), - &in, uid); + h = (struct int_tree_map *) + htab_find_with_hash (gimple_referenced_vars (cfun), &in, uid); gcc_assert (h || uid == 0); if (h) return h->to; @@ -1011,3 +1031,26 @@ get_ref_base_and_extent (tree exp, HOST_WIDE_INT *poffset, return exp; } + + +/* Return memory reference statistics for variable VAR in function FN. + This is computed by alias analysis, but it is not kept + incrementally up-to-date. So, these stats are only accurate if + pass_may_alias has been run recently. If no alias information + exists, this function returns NULL. */ + +mem_sym_stats_t +mem_sym_stats (struct function *fn, tree var) +{ + void **slot; + struct pointer_map_t *stats_map = gimple_mem_ref_stats (fn)->mem_sym_stats; + + if (stats_map == NULL) + return NULL; + + slot = pointer_map_contains (stats_map, var); + if (slot == NULL) + return NULL; + + return (mem_sym_stats_t) *slot; +} diff --git a/gcc/tree-flow-inline.h b/gcc/tree-flow-inline.h index f7728e43ed8..054ddfa28ad 100644 --- a/gcc/tree-flow-inline.h +++ b/gcc/tree-flow-inline.h @@ -871,31 +871,6 @@ memory_partition (tree sym) return tag; } - -/* Set MPT to be the memory partition associated with symbol SYM. */ - -static inline void -set_memory_partition (tree sym, tree mpt) -{ -#if defined ENABLE_CHECKING - if (mpt) - gcc_assert (TREE_CODE (mpt) == MEMORY_PARTITION_TAG - && !is_gimple_reg (sym)); -#endif - var_ann (sym)->mpt = mpt; - if (mpt) - { - bitmap_set_bit (MPT_SYMBOLS (mpt), DECL_UID (sym)); - - /* MPT inherits the call-clobbering attributes from SYM. */ - if (is_call_clobbered (sym)) - { - MTAG_GLOBAL (mpt) = 1; - mark_call_clobbered (mpt, ESCAPE_IS_GLOBAL); - } - } -} - /* Return true if NAME is a memory factoring SSA name (i.e., an SSA name for a memory partition. */ @@ -1810,4 +1785,11 @@ gimple_ssa_operands (struct function *fun) { return &fun->gimple_df->ssa_operands; } + +/* Map describing reference statistics for function FN. */ +static inline struct mem_ref_stats_d * +gimple_mem_ref_stats (struct function *fn) +{ + return &fn->gimple_df->mem_ref_stats; +} #endif /* _TREE_FLOW_INLINE_H */ diff --git a/gcc/tree-flow.h b/gcc/tree-flow.h index ea2d677be43..4d9b36e4ab8 100644 --- a/gcc/tree-flow.h +++ b/gcc/tree-flow.h @@ -41,10 +41,85 @@ typedef struct basic_block_def *basic_block; #endif struct static_var_ann_d; +/* Memory reference statistics for individual memory symbols, + collected during alias analysis. */ +struct mem_sym_stats_d GTY(()) +{ + /* Memory symbol. */ + tree var; + + /* Nonzero if this entry has been assigned a partition. */ + unsigned int partitioned_p : 1; + + /* Nonzero if VAR is a memory partition tag that already contains + call-clobbered variables in its partition set. */ + unsigned int has_call_clobbered_vars : 1; + + /* Number of direct reference sites. A direct reference to VAR is any + reference of the form 'VAR = ' or ' = VAR'. For GIMPLE reg + pointers, this is the number of sites where the pointer is + dereferenced. */ + long num_direct_writes; + long num_direct_reads; + + /* Number of indirect reference sites. An indirect reference to VAR + is any reference via a pointer that contains VAR in its points-to + set or, in the case of call-clobbered symbols, a function call. */ + long num_indirect_writes; + long num_indirect_reads; + + /* Execution frequency. This is the sum of the execution + frequencies of all the statements that reference this object + weighted by the number of references in each statement. This is + the main key used to sort the list of symbols to partition. + Symbols with high execution frequencies are put at the bottom of + the work list (ie, they are partitioned last). + Execution frequencies are taken directly from each basic block, + so compiling with PGO enabled will increase the precision of this + estimate. */ + long frequency_reads; + long frequency_writes; + + /* Set of memory tags that contain VAR in their alias set. */ + bitmap parent_tags; +}; + +typedef struct mem_sym_stats_d *mem_sym_stats_t; +DEF_VEC_P(mem_sym_stats_t); +DEF_VEC_ALLOC_P(mem_sym_stats_t, heap); + +/* Memory reference statistics collected during alias analysis. */ +struct mem_ref_stats_d GTY(()) +{ + /* Number of statements that make memory references. */ + long num_mem_stmts; + + /* Number of statements that make function calls. */ + long num_call_sites; + + /* Number of statements that make calls to pure/const functions. */ + long num_pure_const_call_sites; + + /* Number of ASM statements. */ + long num_asm_sites; + + /* Estimated number of virtual operands needed as computed by + compute_memory_partitions. */ + long num_vuses; + long num_vdefs; + + /* This maps every symbol used to make "memory" references + (pointers, arrays, structures, etc) to an instance of struct + mem_sym_stats_d describing reference statistics for the symbol. */ + struct pointer_map_t * GTY((skip)) mem_sym_stats; +}; + + /* Gimple dataflow datastructure. All publicly available fields shall have gimple_ accessor defined in tree-flow-inline.h, all publicly modifiable fields should have gimple_set accessor. */ -struct gimple_df GTY(()) { +struct gimple_df GTY(()) +{ /* Array of all variables referenced in the function. */ htab_t GTY((param_is (struct int_tree_map))) referenced_vars; @@ -98,6 +173,11 @@ struct gimple_df GTY(()) { /* Hashtable of variables annotations. Used for static variables only; local variables have direct pointer in the tree node. */ htab_t GTY((param_is (struct static_var_ann_d))) var_anns; + + /* Memory reference statistics collected during alias analysis. + This information is used to drive the memory partitioning + heuristics in compute_memory_partitions. */ + struct mem_ref_stats_d mem_ref_stats; }; /* Accessors for internal use only. Generic code should use abstraction @@ -205,6 +285,30 @@ enum need_phi_state { NEED_PHI_STATE_MAYBE }; + +/* The "no alias" attribute allows alias analysis to make more + aggressive assumptions when assigning alias sets, computing + points-to information and memory partitions. These attributes + are the result of user annotations or flags (e.g., + -fargument-noalias). */ +enum noalias_state { + /* Default state. No special assumptions can be made about this + symbol. */ + MAY_ALIAS = 0, + + /* The symbol does not alias with other symbols that have a + NO_ALIAS* attribute. */ + NO_ALIAS, + + /* The symbol does not alias with other symbols that have a + NO_ALIAS*, and it may not alias with global symbols. */ + NO_ALIAS_GLOBAL, + + /* The symbol does not alias with any other symbols. */ + NO_ALIAS_ANYTHING +}; + + struct subvar; typedef struct subvar *subvar_t; @@ -246,12 +350,18 @@ struct var_ann_d GTY(()) in the VDEF list. */ unsigned in_vdef_list : 1; - /* True for HEAP and PARM_NOALIAS artificial variables. */ + /* True for HEAP artificial variables. These variables represent + the memory area allocated by a call to malloc. */ unsigned is_heapvar : 1; /* True if the variable is call clobbered. */ unsigned int call_clobbered : 1; + /* This field describes several "no alias" attributes that some + symbols are known to have. See the enum's definition for more + information on each attribute. */ + ENUM_BITFIELD (noalias_state) noalias_state : 2; + /* Memory partition tag assigned to this symbol. */ tree mpt; @@ -694,10 +804,10 @@ extern void add_referenced_var (tree); extern void remove_referenced_var (tree); extern void mark_symbols_for_renaming (tree); extern void find_new_referenced_vars (tree *); - extern tree make_rename_temp (tree, const char *); extern void set_default_def (tree, tree); extern tree gimple_default_def (struct function *, tree); +extern struct mem_sym_stats_d *mem_sym_stats (struct function *, tree); /* In tree-phinodes.c */ extern void reserve_phi_args_for_new_edge (basic_block); @@ -725,7 +835,8 @@ extern bool may_be_aliased (tree); extern bool is_aliased_with (tree, tree); extern struct ptr_info_def *get_ptr_info (tree); extern void new_type_alias (tree, tree, tree); -extern void count_uses_and_derefs (tree, tree, unsigned *, unsigned *, bool *); +extern void count_uses_and_derefs (tree, tree, unsigned *, unsigned *, + unsigned *); static inline subvar_t get_subvars_for_var (tree); static inline tree get_subvar_at (tree, unsigned HOST_WIDE_INT); static inline bool ref_contains_array_ref (tree); @@ -737,6 +848,12 @@ static inline bool overlap_subvar (unsigned HOST_WIDE_INT, unsigned HOST_WIDE_INT, tree, bool *); extern tree create_tag_raw (enum tree_code, tree, const char *); +extern void delete_mem_ref_stats (struct function *); +extern void dump_mem_ref_stats (FILE *); +extern void debug_mem_ref_stats (void); +extern void debug_memory_partitions (void); +extern void debug_mem_sym_stats (tree var); +extern void debug_all_mem_sym_stats (void); /* Call-back function for walk_use_def_chains(). At each reaching definition, a function with this prototype is called. */ diff --git a/gcc/tree-ssa-alias.c b/gcc/tree-ssa-alias.c index e3cdaf9d590..7fc8f455bcc 100644 --- a/gcc/tree-ssa-alias.c +++ b/gcc/tree-ssa-alias.c @@ -58,21 +58,6 @@ struct alias_map_d }; -/* Data structures used for computing memory partitions. */ - -struct mp_info_def -{ - /* Symbol or memory tag. */ - tree var; - - /* Number of virtual operators needed to represent references to VAR. */ - long num_vops; -}; - -typedef struct mp_info_def *mp_info_t; -DEF_VEC_P(mp_info_t); -DEF_VEC_ALLOC_P(mp_info_t, heap); - /* Counters used to display statistics on alias analysis. */ struct alias_stats_d { @@ -106,13 +91,66 @@ static void delete_alias_info (struct alias_info *); static void compute_flow_sensitive_aliasing (struct alias_info *); static void setup_pointers_and_addressables (struct alias_info *); static void create_global_var (void); -static void maybe_create_global_var (struct alias_info *ai); -static void set_pt_anything (tree ptr); +static void maybe_create_global_var (void); +static void set_pt_anything (tree); -void dump_mp_info (FILE *, VEC(mp_info_t,heap) *mp_info_t); -void debug_mp_info (VEC(mp_info_t,heap) *mp_info_t); +void debug_mp_info (VEC(mem_sym_stats_t,heap) *); + + +/* Return memory reference stats for symbol VAR. Create a new slot in + cfun->gimple_df->mem_sym_stats if needed. */ + +static struct mem_sym_stats_d * +get_mem_sym_stats_for (tree var) +{ + void **slot; + struct mem_sym_stats_d *stats; + struct pointer_map_t *map = gimple_mem_ref_stats (cfun)->mem_sym_stats; + + gcc_assert (map); + + slot = pointer_map_insert (map, var); + if (*slot == NULL) + { + stats = XCNEW (struct mem_sym_stats_d); + stats->var = var; + *slot = (void *) stats; + } + else + stats = (struct mem_sym_stats_d *) *slot; + + return stats; +} + + +/* Set MPT to be the memory partition associated with symbol SYM. */ + +static inline void +set_memory_partition (tree sym, tree mpt) +{ +#if defined ENABLE_CHECKING + if (mpt) + gcc_assert (TREE_CODE (mpt) == MEMORY_PARTITION_TAG + && !is_gimple_reg (sym)); +#endif + + var_ann (sym)->mpt = mpt; + if (mpt) + { + if (MPT_SYMBOLS (mpt) == NULL) + MPT_SYMBOLS (mpt) = BITMAP_ALLOC (&alias_bitmap_obstack); + + bitmap_set_bit (MPT_SYMBOLS (mpt), DECL_UID (sym)); + + /* MPT inherits the call-clobbering attributes from SYM. */ + if (is_call_clobbered (sym)) + { + MTAG_GLOBAL (mpt) = 1; + mark_call_clobbered (mpt, ESCAPE_IS_GLOBAL); + } + } +} -/* Global declarations. */ /* Mark variable VAR as being non-addressable. */ @@ -134,7 +172,15 @@ mark_non_addressable (tree var) if (mpt) { - bitmap_clear_bit (MPT_SYMBOLS (mpt), DECL_UID (var)); + /* Note that it's possible for a symbol to have an associated + MPT and the MPT have a NULL empty set. During + init_alias_info, all MPTs get their sets cleared out, but the + symbols still point to the old MPTs that used to hold them. + This is done so that compute_memory_partitions can now which + symbols are losing or changing partitions and mark them for + renaming. */ + if (MPT_SYMBOLS (mpt)) + bitmap_clear_bit (MPT_SYMBOLS (mpt), DECL_UID (var)); set_memory_partition (var, NULL_TREE); } } @@ -425,174 +471,475 @@ compute_call_clobbered (struct alias_info *ai) compute_tag_properties (); } -/* Dump the MP_INFO array to FILE. */ + +/* Dump memory partition information to FILE. */ + +static void +dump_memory_partitions (FILE *file) +{ + unsigned i, npart; + unsigned long nsyms; + tree mpt; + + fprintf (file, "\nMemory partitions\n\n"); + for (i = 0, npart = 0, nsyms = 0; + VEC_iterate (tree, gimple_ssa_operands (cfun)->mpt_table, i, mpt); + i++) + { + if (mpt) + { + bitmap syms = MPT_SYMBOLS (mpt); + unsigned long n = (syms) ? bitmap_count_bits (syms) : 0; + + fprintf (file, "#%u: ", i); + print_generic_expr (file, mpt, 0); + fprintf (file, ": %lu elements: ", n); + dump_decl_set (file, syms); + npart++; + nsyms += n; + } + } + + fprintf (file, "\n%u memory partitions holding %lu symbols\n", npart, nsyms); +} + + +/* Dump memory partition information to stderr. */ void -dump_mp_info (FILE *file, VEC(mp_info_t,heap) *mp_info) +debug_memory_partitions (void) +{ + dump_memory_partitions (stderr); +} + + +/* Return true if memory partitioning is required given the memory + reference estimates in STATS. */ + +static inline bool +need_to_partition_p (struct mem_ref_stats_d *stats) +{ + long num_vops = stats->num_vuses + stats->num_vdefs; + long avg_vops = CEIL (num_vops, stats->num_mem_stmts); + return (num_vops > (long) MAX_ALIASED_VOPS + && avg_vops > (long) AVG_ALIASED_VOPS); +} + + +/* Count the actual number of virtual operators in CFUN. Note that + this is only meaningful after virtual operands have been populated, + so it should be invoked at the end of compute_may_aliases. + + The number of virtual operators are stored in *NUM_VDEFS_P and + *NUM_VUSES_P, the number of partitioned symbols in + *NUM_PARTITIONED_P and the number of unpartitioned symbols in + *NUM_UNPARTITIONED_P. + + If any of these pointers is NULL the corresponding count is not + computed. */ + +static void +count_mem_refs (long *num_vuses_p, long *num_vdefs_p, + long *num_partitioned_p, long *num_unpartitioned_p) +{ + block_stmt_iterator bsi; + basic_block bb; + long num_vdefs, num_vuses, num_partitioned, num_unpartitioned; + referenced_var_iterator rvi; + tree sym; + + num_vuses = num_vdefs = num_partitioned = num_unpartitioned = 0; + + if (num_vuses_p || num_vdefs_p) + FOR_EACH_BB (bb) + for (bsi = bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi)) + { + tree stmt = bsi_stmt (bsi); + if (stmt_references_memory_p (stmt)) + { + num_vuses += NUM_SSA_OPERANDS (stmt, SSA_OP_VUSE); + num_vdefs += NUM_SSA_OPERANDS (stmt, SSA_OP_VDEF); + } + } + + if (num_partitioned_p || num_unpartitioned_p) + FOR_EACH_REFERENCED_VAR (sym, rvi) + { + if (is_gimple_reg (sym)) + continue; + + if (memory_partition (sym)) + num_partitioned++; + else + num_unpartitioned++; + } + + if (num_vdefs_p) + *num_vdefs_p = num_vdefs; + + if (num_vuses_p) + *num_vuses_p = num_vuses; + + if (num_partitioned_p) + *num_partitioned_p = num_partitioned; + + if (num_unpartitioned_p) + *num_unpartitioned_p = num_unpartitioned; +} + + +/* Dump memory reference stats for function CFUN to FILE. */ + +void +dump_mem_ref_stats (FILE *file) +{ + long actual_num_vuses, actual_num_vdefs; + long num_partitioned, num_unpartitioned; + struct mem_ref_stats_d *stats; + + stats = gimple_mem_ref_stats (cfun); + + count_mem_refs (&actual_num_vuses, &actual_num_vdefs, &num_partitioned, + &num_unpartitioned); + + fprintf (file, "\nMemory reference statistics for %s\n\n", + lang_hooks.decl_printable_name (current_function_decl, 2)); + + fprintf (file, "Number of memory statements: %ld\n", + stats->num_mem_stmts); + fprintf (file, "Number of call sites: %ld\n", + stats->num_call_sites); + fprintf (file, "Number of pure/const call sites: %ld\n", + stats->num_pure_const_call_sites); + fprintf (file, "Number of asm sites: %ld\n", + stats->num_asm_sites); + fprintf (file, "Estimated number of loads: %ld (%ld/stmt)\n", + stats->num_vuses, + (stats->num_mem_stmts) + ? CEIL (stats->num_vuses, stats->num_mem_stmts) + : 0); + fprintf (file, "Actual number of loads: %ld (%ld/stmt)\n", + actual_num_vuses, + (stats->num_mem_stmts) + ? CEIL (actual_num_vuses, stats->num_mem_stmts) + : 0); + + if (actual_num_vuses > stats->num_vuses + (stats->num_vuses / 25)) + fprintf (file, "\t(warning: estimation is lower by more than 25%%)\n"); + + fprintf (file, "Estimated number of stores: %ld (%ld/stmt)\n", + stats->num_vdefs, + (stats->num_mem_stmts) + ? CEIL (stats->num_vdefs, stats->num_mem_stmts) + : 0); + fprintf (file, "Actual number of stores: %ld (%ld/stmt)\n", + actual_num_vdefs, + (stats->num_mem_stmts) + ? CEIL (actual_num_vdefs, stats->num_mem_stmts) + : 0); + + if (actual_num_vdefs > stats->num_vdefs + (stats->num_vdefs / 25)) + fprintf (file, "\t(warning: estimation is lower by more than 25%%)\n"); + + fprintf (file, "Partitioning thresholds: MAX = %d AVG = %d " + "(%sNEED TO PARTITION)\n", MAX_ALIASED_VOPS, AVG_ALIASED_VOPS, + need_to_partition_p (stats) ? "" : "NO "); + fprintf (file, "Number of partitioned symbols: %ld\n", num_partitioned); + fprintf (file, "Number of unpartitioned symbols: %ld\n", num_unpartitioned); +} + + +/* Dump memory reference stats for function FN to stderr. */ + +void +debug_mem_ref_stats (void) +{ + dump_mem_ref_stats (stderr); +} + + +/* Dump memory reference stats for variable VAR to FILE. */ + +static void +dump_mem_sym_stats (FILE *file, tree var) +{ + mem_sym_stats_t stats = mem_sym_stats (cfun, var); + + if (stats == NULL) + return; + + fprintf (file, "read frequency: %6ld, write frequency: %6ld, " + "direct reads: %3ld, direct writes: %3ld, " + "indirect reads: %4ld, indirect writes: %4ld, symbol: ", + stats->frequency_reads, stats->frequency_writes, + stats->num_direct_reads, stats->num_direct_writes, + stats->num_indirect_reads, stats->num_indirect_writes); + print_generic_expr (file, stats->var, 0); + fprintf (file, ", tags: "); + dump_decl_set (file, stats->parent_tags); +} + + +/* Dump memory reference stats for variable VAR to stderr. */ + +void +debug_mem_sym_stats (tree var) +{ + dump_mem_sym_stats (stderr, var); +} + + +/* Dump memory reference stats for all memory symbols to FILE. */ + +static void +dump_all_mem_sym_stats (FILE *file) +{ + referenced_var_iterator rvi; + tree sym; + + FOR_EACH_REFERENCED_VAR (sym, rvi) + { + if (is_gimple_reg (sym)) + continue; + + dump_mem_sym_stats (file, sym); + } +} + + +/* Dump memory reference stats for all memory symbols to stderr. */ + +void +debug_all_mem_sym_stats (void) +{ + dump_all_mem_sym_stats (stderr); +} + + +/* Dump the MP_INFO array to FILE. */ + +static void +dump_mp_info (FILE *file, VEC(mem_sym_stats_t,heap) *mp_info) { unsigned i; - mp_info_t mp_p; + mem_sym_stats_t mp_p; - for (i = 0; VEC_iterate (mp_info_t, mp_info, i, mp_p); i++) - { - fprintf (file, "%6lu\t", mp_p->num_vops); - if (mp_p->var == NULL_TREE) - { - fprintf (file, "CALL-CLOBBERED SYMBOLS: "); - dump_decl_set (file, gimple_call_clobbered_vars (cfun)); - } - else - dump_variable (file, mp_p->var); - } + for (i = 0; VEC_iterate (mem_sym_stats_t, mp_info, i, mp_p); i++) + if (!mp_p->partitioned_p) + dump_mem_sym_stats (file, mp_p->var); } /* Dump the MP_INFO array to stderr. */ void -debug_mp_info (VEC(mp_info_t,heap) *mp_info) +debug_mp_info (VEC(mem_sym_stats_t,heap) *mp_info) { dump_mp_info (stderr, mp_info); } -/* Comparison function for qsort used in sort_mp_info. */ +/* Update memory reference stats for symbol VAR in statement STMT. + NUM_DIRECT_READS and NUM_DIRECT_WRITES specify the number of times + that VAR is read/written in STMT (indirect reads/writes are not + recorded by this function, see compute_memory_partitions). */ -static int -mp_info_cmp (const void *p, const void *q) +void +update_mem_sym_stats_from_stmt (tree var, tree stmt, long num_direct_reads, + long num_direct_writes) { - mp_info_t e1 = *((const mp_info_t *) p); - mp_info_t e2 = *((const mp_info_t *) q); + mem_sym_stats_t stats; - /* We want to sort in decreasing order. */ - if (e1->num_vops < e2->num_vops) - return 1; - else if (e1->num_vops > e2->num_vops) + gcc_assert (num_direct_reads >= 0 && num_direct_writes >= 0); + + stats = get_mem_sym_stats_for (var); + + stats->num_direct_reads += num_direct_reads; + stats->frequency_reads += ((long) bb_for_stmt (stmt)->frequency + * num_direct_reads); + + stats->num_direct_writes += num_direct_writes; + stats->frequency_writes += ((long) bb_for_stmt (stmt)->frequency + * num_direct_writes); +} + + +/* The list is sorted by increasing partitioning score (PSCORE). + This score is computed such that symbols with high scores are + those that are least likely to be partitioned. Given a symbol + MP->VAR, PSCORE(S) is the result of the following weighted sum + + PSCORE(S) = FW * 64 + FR * 32 + + DW * 16 + DR * 8 + + IW * 4 + IR * 2 + + NO_ALIAS + + where + + FW Execution frequency of writes to S + FR Execution frequency of reads from S + DW Number of direct writes to S + DR Number of direct reads from S + IW Number of indirect writes to S + IR Number of indirect reads from S + NO_ALIAS State of the NO_ALIAS* flags + + The basic idea here is that symbols that are frequently + written-to in hot paths of the code are the last to be considered + for partitioning. */ + +static inline long +pscore (mem_sym_stats_t mp) +{ + return mp->frequency_writes * 64 + mp->frequency_reads * 32 + + mp->num_direct_writes * 16 + mp->num_direct_reads * 8 + + mp->num_indirect_writes * 4 + mp->num_indirect_reads * 2 + + var_ann (mp->var)->noalias_state; +} + + +/* Given two MP_INFO entries MP1 and MP2, return -1 if MP1->VAR should + be partitioned before MP2->VAR, 0 if they are the same or 1 if + MP1->VAR should be partitioned after MP2->VAR. */ + +static inline int +compare_mp_info_entries (mem_sym_stats_t mp1, mem_sym_stats_t mp2) +{ + long pscore1 = pscore (mp1); + long pscore2 = pscore (mp2); + + if (pscore1 < pscore2) return -1; + else if (pscore1 > pscore2) + return 1; else return 0; } +/* Comparison routine for qsort. The list is sorted by increasing + partitioning score (PSCORE). This score is computed such that + symbols with high scores are those that are least likely to be + partitioned. */ + +static int +mp_info_cmp (const void *p, const void *q) +{ + mem_sym_stats_t e1 = *((const mem_sym_stats_t *) p); + mem_sym_stats_t e2 = *((const mem_sym_stats_t *) q); + return compare_mp_info_entries (e1, e2); +} + + /* Sort the array of reference counts used to compute memory partitions. - Elements are sorted in descending order of virtual operators needed. */ + Elements are sorted in ascending order of execution frequency and + descending order of virtual operators needed. */ static inline void -sort_mp_info (VEC(mp_info_t,heap) *list) +sort_mp_info (VEC(mem_sym_stats_t,heap) *list) { - unsigned num = VEC_length (mp_info_t, list); + unsigned num = VEC_length (mem_sym_stats_t, list); if (num < 2) return; if (num == 2) { - if (VEC_index (mp_info_t, list, 0)->num_vops - < VEC_index (mp_info_t, list, 1)->num_vops) + if (compare_mp_info_entries (VEC_index (mem_sym_stats_t, list, 0), + VEC_index (mem_sym_stats_t, list, 1)) > 0) { /* Swap elements if they are in the wrong order. */ - mp_info_t tmp = VEC_index (mp_info_t, list, 0); - VEC_replace (mp_info_t, list, 0, VEC_index (mp_info_t, list, 1)); - VEC_replace (mp_info_t, list, 1, tmp); + mem_sym_stats_t tmp = VEC_index (mem_sym_stats_t, list, 0); + VEC_replace (mem_sym_stats_t, list, 0, + VEC_index (mem_sym_stats_t, list, 1)); + VEC_replace (mem_sym_stats_t, list, 1, tmp); } return; } /* There are 3 or more elements, call qsort. */ - qsort (VEC_address (mp_info_t, list), VEC_length (mp_info_t, list), - sizeof (mp_info_t), mp_info_cmp); + qsort (VEC_address (mem_sym_stats_t, list), + VEC_length (mem_sym_stats_t, list), + sizeof (mem_sym_stats_t), + mp_info_cmp); } -/* Create a new partition to hold all the symbols aliased with - MP_P->VAR. If MP_P->VAR is NULL, it partitions the call-clobbered - variables. Only symbols that are not already in another partition - are added to the new partition created for MP_P->VAR. */ +/* Return the memory partition tag (MPT) associated with memory + symbol SYM. */ -static void -create_partition_for (mp_info_t mp_p) +static tree +get_mpt_for (tree sym) +{ + tree mpt; + + /* Don't create a new tag unnecessarily. */ + mpt = memory_partition (sym); + if (mpt == NULL_TREE) + { + mpt = create_tag_raw (MEMORY_PARTITION_TAG, TREE_TYPE (sym), "MPT"); + TREE_ADDRESSABLE (mpt) = 0; + add_referenced_var (mpt); + VEC_safe_push (tree, heap, gimple_ssa_operands (cfun)->mpt_table, mpt); + gcc_assert (MPT_SYMBOLS (mpt) == NULL); + set_memory_partition (sym, mpt); + } + + return mpt; +} + + +/* Add MP_P->VAR to a memory partition and return the partition. */ + +static tree +find_partition_for (mem_sym_stats_t mp_p) { - bitmap_iterator bi; - tree mpt, sym; - bitmap aliases; unsigned i; + VEC(tree,heap) *mpt_table; + tree mpt; - if (mp_p->num_vops <= (long) MAX_ALIASED_VOPS) - return; + mpt_table = gimple_ssa_operands (cfun)->mpt_table; + mpt = NULL_TREE; - if (mp_p->var == NULL_TREE) + /* Find an existing partition for MP_P->VAR. */ + for (i = 0; VEC_iterate (tree, mpt_table, i, mpt); i++) { - bitmap_iterator bi; - bitmap tmp; + mem_sym_stats_t mpt_stats; - /* Since the partitions we create for call-clobbered variables - will also be marked call-clobbered, make a copy of the - original set to avoid confusing the iterator. */ - tmp = BITMAP_ALLOC (NULL); - bitmap_copy (tmp, gimple_call_clobbered_vars (cfun)); + /* If MPT does not have any symbols yet, use it. */ + if (MPT_SYMBOLS (mpt) == NULL) + break; - /* Process call-clobbered symbols when no MP_P->VAR is given. */ - mpt = NULL_TREE; - EXECUTE_IF_SET_IN_BITMAP (tmp, 0, i, bi) - { - tree sym = referenced_var (i); - if (memory_partition (sym) == NULL_TREE) - { - if (mpt == NULL_TREE) - { - mpt = get_mpt_for (sym); - mp_p->num_vops++; - } + /* Otherwise, see if MPT has common parent tags with MP_P->VAR, + but avoid grouping clobbered variables with non-clobbered + variables (otherwise, this tends to creates a single memory + partition because other call-clobbered variables may have + common parent tags with non-clobbered ones). */ + mpt_stats = get_mem_sym_stats_for (mpt); + if (mp_p->parent_tags + && mpt_stats->parent_tags + && is_call_clobbered (mpt) == is_call_clobbered (mp_p->var) + && bitmap_intersect_p (mpt_stats->parent_tags, mp_p->parent_tags)) + break; - mark_sym_for_renaming (mpt); - mark_sym_for_renaming (sym); - set_memory_partition (sym, mpt); - } - - mp_p->num_vops--; - - /* If we have already grouped enough, stop. */ - if (mp_p->num_vops <= (long) MAX_ALIASED_VOPS) - break; - } - - BITMAP_FREE (tmp); + /* If no common parent tags are found, see if both MPT and + MP_P->VAR are call-clobbered. */ + if (is_call_clobbered (mpt) && is_call_clobbered (mp_p->var)) + break; } + + if (mpt == NULL_TREE) + mpt = get_mpt_for (mp_p->var); else - { - aliases = may_aliases (mp_p->var); - gcc_assert (!bitmap_empty_p (aliases)); + set_memory_partition (mp_p->var, mpt); - mpt = NULL_TREE; - EXECUTE_IF_SET_IN_BITMAP (aliases, 0, i, bi) - { - sym = referenced_var (i); - /* Only set the memory partition for aliased symbol SYM if - SYM does not belong to another partition. */ - if (memory_partition (sym) == NULL_TREE) - { - if (mpt == NULL_TREE) - { - mpt = get_mpt_for (mp_p->var); - mp_p->num_vops++; - } + mp_p->partitioned_p = true; - mark_sym_for_renaming (mpt); - mark_sym_for_renaming (sym); - set_memory_partition (sym, mpt); - } + mark_sym_for_renaming (mp_p->var); + mark_sym_for_renaming (mpt); - mp_p->num_vops--; - - /* If we have already grouped enough, stop. */ - if (mp_p->num_vops <= (long) MAX_ALIASED_VOPS) - break; - } - - if (mpt) - mark_call_clobbered (mpt, ESCAPE_UNKNOWN); - } + return mpt; } @@ -608,153 +955,432 @@ rewrite_alias_set_for (tree tag, bitmap new_aliases) unsigned i; tree mpt, sym; - if (tag == NULL_TREE) + EXECUTE_IF_SET_IN_BITMAP (MTAG_ALIASES (tag), 0, i, bi) { - /* Do not rewrite CALL_CLOBBERED_VARS. If a symbol S is taken - out of this set, the optimizers will no longer consider S as - call-clobbered, and that may lead to wrong transformations - (e.g., pass_tail_calls explicitly examines all the symbols in - the function to determine if it should enable tail-call - marking). */ - return; + sym = referenced_var (i); + mpt = memory_partition (sym); + if (mpt) + bitmap_set_bit (new_aliases, DECL_UID (mpt)); + else + bitmap_set_bit (new_aliases, DECL_UID (sym)); } - else + + /* Rebuild the may-alias array for TAG. */ + bitmap_copy (MTAG_ALIASES (tag), new_aliases); +} + + +/* Determine how many virtual operands can be saved by partitioning + MP_P->VAR into MPT. When a symbol S is thrown inside a partition + P, every virtual operand that used to reference S will now + reference P. Whether it reduces the number of virtual operands + depends on: + + 1- Direct references to S are never saved. Instead of the virtual + operand to S, we will now have a virtual operand to P. + + 2- Indirect references to S are reduced only for those memory tags + holding S that already had other symbols partitioned into P. + For instance, if a memory tag T has the alias set { a b S c }, + the first time we partition S into P, the alias set will become + { a b P c }, so no virtual operands will be saved. However, if + we now partition symbol 'c' into P, then the alias set for T + will become { a b P }, so we will be saving one virtual operand + for every indirect reference to 'c'. + + 3- Is S is call-clobbered, we save as many virtual operands as + call/asm sites exist in the code, but only if other + call-clobbered symbols have been grouped into P. The first + call-clobbered symbol that we group does not produce any + savings. + + MEM_REF_STATS points to CFUN's memory reference information. */ + +static void +estimate_vop_reduction (struct mem_ref_stats_d *mem_ref_stats, + mem_sym_stats_t mp_p, tree mpt) +{ + unsigned i; + bitmap_iterator bi; + mem_sym_stats_t mpt_stats; + + /* We should only get symbols with indirect references here. */ + gcc_assert (mp_p->num_indirect_reads > 0 || mp_p->num_indirect_writes > 0); + + /* Note that the only statistics we keep for MPT is the set of + parent tags to know which memory tags have had alias members + partitioned, and the indicator has_call_clobbered_vars. + Reference counts are not important for MPT. */ + mpt_stats = get_mem_sym_stats_for (mpt); + + /* Traverse all the parent tags for MP_P->VAR. For every tag T, if + partition P is already grouping aliases of T, then reduce the + number of virtual operands by the number of direct references + to T. */ + if (mp_p->parent_tags) { - /* Create a new alias set for TAG with the new partitions. */ + if (mpt_stats->parent_tags == NULL) + mpt_stats->parent_tags = BITMAP_ALLOC (&alias_bitmap_obstack); - EXECUTE_IF_SET_IN_BITMAP (MTAG_ALIASES (tag), 0, i, bi) + EXECUTE_IF_SET_IN_BITMAP (mp_p->parent_tags, 0, i, bi) { - sym = referenced_var (i); - mpt = memory_partition (sym); - if (mpt) - bitmap_set_bit (new_aliases, DECL_UID (mpt)); + if (bitmap_bit_p (mpt_stats->parent_tags, i)) + { + /* Partition MPT is already partitioning symbols in the + alias set for TAG. This means that we are now saving + 1 virtual operand for every direct reference to TAG. */ + tree tag = referenced_var (i); + mem_sym_stats_t tag_stats = mem_sym_stats (cfun, tag); + mem_ref_stats->num_vuses -= tag_stats->num_direct_reads; + mem_ref_stats->num_vdefs -= tag_stats->num_direct_writes; + } else - bitmap_set_bit (new_aliases, DECL_UID (sym)); + { + /* This is the first symbol in tag I's alias set that is + being grouped under MPT. We will not save any + virtual operands this time, but record that MPT is + grouping a symbol from TAG's alias set so that the + next time we get the savings. */ + bitmap_set_bit (mpt_stats->parent_tags, i); + } } + } - /* Rebuild the may-alias array for TAG. */ - bitmap_copy (MTAG_ALIASES (tag), new_aliases); + /* If MP_P->VAR is call-clobbered, and MPT is already grouping + call-clobbered symbols, then we will save as many virtual + operands as asm/call sites there are. */ + if (is_call_clobbered (mp_p->var)) + { + if (mpt_stats->has_call_clobbered_vars) + mem_ref_stats->num_vdefs -= mem_ref_stats->num_call_sites + + mem_ref_stats->num_asm_sites; + else + mpt_stats->has_call_clobbered_vars = true; } } -/* Compute memory partitions. +/* Helper for compute_memory_partitions. Transfer reference counts + from pointers to their pointed-to sets. Counters for pointers were + computed by update_alias_info. MEM_REF_STATS points to CFUN's + memory reference information. */ - The partitioning is straightforward: +static void +update_reference_counts (struct mem_ref_stats_d *mem_ref_stats) +{ + unsigned i; + bitmap_iterator bi; + mem_sym_stats_t sym_stats; + + for (i = 1; i < num_ssa_names; i++) + { + tree ptr; + struct ptr_info_def *pi; + + ptr = ssa_name (i); + if (ptr + && POINTER_TYPE_P (TREE_TYPE (ptr)) + && (pi = SSA_NAME_PTR_INFO (ptr)) != NULL + && pi->is_dereferenced) + { + unsigned j; + bitmap_iterator bj; + tree tag; + mem_sym_stats_t ptr_stats, tag_stats; + + /* If PTR has flow-sensitive points-to information, use + PTR's name tag, otherwise use the symbol tag associated + with PTR's symbol. */ + if (pi->name_mem_tag) + tag = pi->name_mem_tag; + else + tag = symbol_mem_tag (SSA_NAME_VAR (ptr)); + + ptr_stats = get_mem_sym_stats_for (ptr); + tag_stats = get_mem_sym_stats_for (tag); + + /* TAG has as many direct references as dereferences we + found for its parent pointer. */ + tag_stats->num_direct_reads += ptr_stats->num_direct_reads; + tag_stats->num_direct_writes += ptr_stats->num_direct_writes; + + /* All the dereferences of pointer PTR are considered direct + references to PTR's memory tag (TAG). In turn, + references to TAG will become virtual operands for every + symbol in TAG's alias set. So, for every symbol ALIAS in + TAG's alias set, add as many indirect references to ALIAS + as direct references there are for TAG. */ + if (MTAG_ALIASES (tag)) + EXECUTE_IF_SET_IN_BITMAP (MTAG_ALIASES (tag), 0, j, bj) + { + tree alias = referenced_var (j); + sym_stats = get_mem_sym_stats_for (alias); + + /* All the direct references to TAG are indirect references + to ALIAS. */ + sym_stats->num_indirect_reads += ptr_stats->num_direct_reads; + sym_stats->num_indirect_writes += ptr_stats->num_direct_writes; + sym_stats->frequency_reads += ptr_stats->frequency_reads; + sym_stats->frequency_writes += ptr_stats->frequency_writes; + + /* Indicate that TAG is one of ALIAS's parent tags. */ + if (sym_stats->parent_tags == NULL) + sym_stats->parent_tags = BITMAP_ALLOC (&alias_bitmap_obstack); + bitmap_set_bit (sym_stats->parent_tags, DECL_UID (tag)); + } + } + } + + /* Call-clobbered symbols are indirectly written at every + call/asm site. */ + EXECUTE_IF_SET_IN_BITMAP (gimple_call_clobbered_vars (cfun), 0, i, bi) + { + tree sym = referenced_var (i); + sym_stats = get_mem_sym_stats_for (sym); + sym_stats->num_indirect_writes += mem_ref_stats->num_call_sites + + mem_ref_stats->num_asm_sites; + } + + /* Addressable symbols are indirectly written at some ASM sites. + Since only ASM sites that clobber memory actually affect + addressable symbols, this is an over-estimation. */ + EXECUTE_IF_SET_IN_BITMAP (gimple_addressable_vars (cfun), 0, i, bi) + { + tree sym = referenced_var (i); + sym_stats = get_mem_sym_stats_for (sym); + sym_stats->num_indirect_writes += mem_ref_stats->num_asm_sites; + } +} + + +/* Helper for compute_memory_partitions. Add all memory symbols to + *MP_INFO_P and compute the initial estimate for the total number of + virtual operands needed. MEM_REF_STATS points to CFUN's memory + reference information. On exit, *TAGS_P will contain the list of + memory tags whose alias set need to be rewritten after + partitioning. */ + +static void +build_mp_info (struct mem_ref_stats_d *mem_ref_stats, + VEC(mem_sym_stats_t,heap) **mp_info_p, + VEC(tree,heap) **tags_p) +{ + tree var; + referenced_var_iterator rvi; + + FOR_EACH_REFERENCED_VAR (var, rvi) + { + mem_sym_stats_t sym_stats; + tree old_mpt; + + /* We are only interested in memory symbols other than MPTs. */ + if (is_gimple_reg (var) || TREE_CODE (var) == MEMORY_PARTITION_TAG) + continue; + + /* Collect memory tags into the TAGS array so that we can + rewrite their alias sets after partitioning. */ + if (MTAG_P (var) && MTAG_ALIASES (var)) + VEC_safe_push (tree, heap, *tags_p, var); + + /* Since we are going to re-compute partitions, any symbols that + used to belong to a partition must be detached from it and + marked for renaming. */ + if ((old_mpt = memory_partition (var)) != NULL) + { + mark_sym_for_renaming (old_mpt); + set_memory_partition (var, NULL_TREE); + mark_sym_for_renaming (var); + } + + sym_stats = get_mem_sym_stats_for (var); + + /* Add VAR's reference info to MP_INFO. Note that the only + symbols that make sense to partition are those that have + indirect references. If a symbol S is always directly + referenced, partitioning it will not reduce the number of + virtual operators. The only symbols that are profitable to + partition are those that belong to alias sets and/or are + call-clobbered. */ + if (sym_stats->num_indirect_reads > 0 + || sym_stats->num_indirect_writes > 0) + VEC_safe_push (mem_sym_stats_t, heap, *mp_info_p, sym_stats); + + /* Update the number of estimated VOPS. Note that direct + references to memory tags are always counted as indirect + references to their alias set members, so if a memory tag has + aliases, do not count its direct references to avoid double + accounting. */ + if (!MTAG_P (var) || !MTAG_ALIASES (var)) + { + mem_ref_stats->num_vuses += sym_stats->num_direct_reads; + mem_ref_stats->num_vdefs += sym_stats->num_direct_writes; + } + + mem_ref_stats->num_vuses += sym_stats->num_indirect_reads; + mem_ref_stats->num_vdefs += sym_stats->num_indirect_writes; + } +} + + +/* Compute memory partitions. A memory partition (MPT) is an + arbitrary grouping of memory symbols, such that references to one + member of the group is considered a reference to all the members of + the group. - 1- All the memory tags and call-clobbered that cause virtual - operators are collected into the MP_INFO table together with the - number of virtual operands that would be needed to represent all - the members in the alias set. + As opposed to alias sets in memory tags, the grouping into + partitions is completely arbitrary and only done to reduce the + number of virtual operands. The only rule that needs to be + observed when creating memory partitions is that given two memory + partitions MPT.i and MPT.j, they must not contain symbols in + common. - 2- MP_INFO is sorted in decreasing order of virtual operators. + Memory partitions are used when putting the program into Memory-SSA + form. In particular, in Memory-SSA PHI nodes are not computed for + individual memory symbols. They are computed for memory + partitions. This reduces the amount of PHI nodes in the SSA graph + at the expense of precision (i.e., it makes unrelated stores affect + each other). + + However, it is possible to increase precision by changing this + partitioning scheme. For instance, if the partitioning scheme is + such that get_mpt_for is the identity function (that is, + get_mpt_for (s) = s), this will result in ultimate precision at the + expense of huge SSA webs. - 3- For every memory tag T in MP_INFO, a new partition MP is created. + At the other extreme, a partitioning scheme that groups all the + symbols in the same set results in minimal SSA webs and almost + total loss of precision. - 4- All the symbols S in T's alias set are examined. If S is not - already in another partition then S is added to partition MP. + There partitioning heuristic uses three parameters to decide the + order in which symbols are processed. The list of symbols is + sorted so that symbols that are more likely to be partitioned are + near the top of the list: - 6- The estimate of VOPS is updated, if it falls below - MAX_ALIASED_VOPS, we stop. */ + - Execution frequency. If a memory references is in a frequently + executed code path, grouping it into a partition may block useful + transformations and cause sub-optimal code generation. So, the + partition heuristic tries to avoid grouping symbols with high + execution frequency scores. Execution frequency is taken + directly from the basic blocks where every reference is made (see + update_mem_sym_stats_from_stmt), which in turn uses the + profile guided machinery, so if the program is compiled with PGO + enabled, more accurate partitioning decisions will be made. + + - Number of references. Symbols with few references in the code, + are partitioned before symbols with many references. + + - NO_ALIAS attributes. Symbols with any of the NO_ALIAS* + attributes are partitioned after symbols marked MAY_ALIAS. + + Once the list is sorted, the partitioning proceeds as follows: + + 1- For every symbol S in MP_INFO, create a new memory partition MP, + if necessary. To avoid memory partitions that contain symbols + from non-conflicting alias sets, memory partitions are + associated to the memory tag that holds S in its alias set. So, + when looking for a memory partition for S, the memory partition + associated with one of the memory tags holding S is chosen. If + none exists, a new one is created. + + 2- Add S to memory partition MP. + + 3- Reduce by 1 the number of VOPS for every memory tag holding S. + + 4- If the total number of VOPS is less than MAX_ALIASED_VOPS or the + average number of VOPS per statement is less than + AVG_ALIASED_VOPS, stop. Otherwise, go to the next symbol in the + list. */ static void compute_memory_partitions (void) { - referenced_var_iterator rvi; - tree var; + tree tag; unsigned i; - struct mp_info_def mp; - mp_info_t mp_p; - VEC(mp_info_t,heap) *mp_info; - long max_num_vops = 0; + mem_sym_stats_t mp_p; + VEC(mem_sym_stats_t,heap) *mp_info; bitmap new_aliases; + VEC(tree,heap) *tags; + struct mem_ref_stats_d *mem_ref_stats; + int prev_max_aliased_vops; + + mem_ref_stats = gimple_mem_ref_stats (cfun); + gcc_assert (mem_ref_stats->num_vuses == 0 && mem_ref_stats->num_vdefs == 0); + + if (mem_ref_stats->num_mem_stmts == 0) + return; timevar_push (TV_MEMORY_PARTITIONING); mp_info = NULL; - max_num_vops = 0; + tags = NULL; + prev_max_aliased_vops = MAX_ALIASED_VOPS; - /* Add reference counts for all the call-clobbered variables. */ - if (!bitmap_empty_p (gimple_call_clobbered_vars (cfun))) - { - mp.var = NULL_TREE; - mp.num_vops = bitmap_count_bits (gimple_call_clobbered_vars (cfun)); - max_num_vops = mp.num_vops; - mp_p = xcalloc (1, sizeof (*mp_p)); - *mp_p = mp; - VEC_safe_push (mp_info_t, heap, mp_info, mp_p); - } + /* Since we clearly cannot lower the number of virtual operators + below the total number of memory statements in the function, we + may need to adjust MAX_ALIASED_VOPS beforehand. */ + if (MAX_ALIASED_VOPS < mem_ref_stats->num_mem_stmts) + MAX_ALIASED_VOPS = mem_ref_stats->num_mem_stmts; - /* Add reference counts for all the symbol tags. */ - FOR_EACH_REFERENCED_VAR (var, rvi) - { - if (TREE_CODE (var) != SYMBOL_MEMORY_TAG - && TREE_CODE (var) != NAME_MEMORY_TAG) - continue; + /* Update reference stats for all the pointed-to variables and + memory tags. */ + update_reference_counts (mem_ref_stats); - /* Each reference to VAR will produce as many VOPs as elements - exist in its alias set. */ - mp.var = var; - if (!may_aliases (var)) - mp.num_vops = 0; - else - mp.num_vops = bitmap_count_bits (may_aliases (var)); - - /* No point grouping singleton alias sets. */ - if (mp.num_vops <= 1) - continue; - - mp_p = xcalloc (1, sizeof (*mp_p)); - *mp_p = mp; - VEC_safe_push (mp_info_t, heap, mp_info, mp_p); - - if (mp.num_vops > max_num_vops) - max_num_vops = mp.num_vops; - } - - if (dump_file) - { - fprintf (dump_file, "\n%s: Maximum number of VOPS needed per statement: " - "%ld\n", get_name (current_function_decl), max_num_vops); - } + /* Add all the memory symbols to MP_INFO. */ + build_mp_info (mem_ref_stats, &mp_info, &tags); /* No partitions required if we are below the threshold. */ - if (max_num_vops <= (long) MAX_ALIASED_VOPS) - goto done; + if (!need_to_partition_p (mem_ref_stats)) + { + if (dump_file) + fprintf (dump_file, "\nMemory partitioning NOT NEEDED for %s\n", + get_name (current_function_decl)); + goto done; + } - /* Sort the MP_INFO array in order of decreasing number of - virtual operands. */ + /* Sort the MP_INFO array so that symbols that should be partitioned + first are near the top of the list. */ sort_mp_info (mp_info); if (dump_file) { - fprintf (dump_file, "\nVOPS generated by pointer dereferences " - "before partitioning:\n"); + fprintf (dump_file, "\nMemory partitioning NEEDED for %s\n\n", + get_name (current_function_decl)); + fprintf (dump_file, "Memory symbol references before partitioning:\n"); dump_mp_info (dump_file, mp_info); } - /* Now that we have all the VOP generating tags in the MP_INFO array - sorted by decreasing number of VOPS, create memory partitions and - group aliased symbols into those partitions. */ - for (i = 0; VEC_iterate (mp_info_t, mp_info, i, mp_p); i++) + /* Create partitions for variables in MP_INFO until we have enough + to lower the total number of VOPS below MAX_ALIASED_VOPS or if + the average number of VOPS per statement is below + AVG_ALIASED_VOPS. */ + for (i = 0; VEC_iterate (mem_sym_stats_t, mp_info, i, mp_p); i++) { - /* Stop processing if we are already below the threshold. */ - if (mp_p->num_vops <= (long) MAX_ALIASED_VOPS) + tree mpt; + + /* If we are below the threshold, stop. */ + if (!need_to_partition_p (mem_ref_stats)) break; - create_partition_for (mp_p); + mpt = find_partition_for (mp_p); + estimate_vop_reduction (mem_ref_stats, mp_p, mpt); } /* After partitions have been created, rewrite alias sets to use them instead of the original symbols. This way, if the alias set was computed as { a b c d e f }, and the subset { b e f } was grouped into partition MPT.3, then the new alias set for the tag - will be { a c d MPT.3 }. */ + will be { a c d MPT.3 }. + + Note that this is not strictly necessary. The operand scanner + will always check if a symbol belongs to a partition when adding + virtual operands. However, by reducing the size of the alias + sets to be scanned, the work needed inside the operand scanner is + significantly reduced. */ new_aliases = BITMAP_ALLOC (NULL); - for (i = 0; VEC_iterate (mp_info_t, mp_info, i, mp_p); i++) + for (i = 0; VEC_iterate (tree, tags, i, tag); i++) { - rewrite_alias_set_for (mp_p->var, new_aliases); + rewrite_alias_set_for (tag, new_aliases); bitmap_clear (new_aliases); } @@ -762,16 +1388,16 @@ compute_memory_partitions (void) if (dump_file) { - fprintf (dump_file, "\nVOPS generated by pointer dereferences " - "after partitioning:\n"); + fprintf (dump_file, "\nMemory symbol references after partitioning:\n"); dump_mp_info (dump_file, mp_info); } done: /* Free allocated memory. */ - for (i = 0; VEC_iterate (mp_info_t, mp_info, i, mp_p); i++) - free (mp_p); - VEC_free (mp_info_t, heap, mp_info); + VEC_free (mem_sym_stats_t, heap, mp_info); + VEC_free (tree, heap, tags); + + MAX_ALIASED_VOPS = prev_max_aliased_vops; timevar_pop (TV_MEMORY_PARTITIONING); } @@ -882,9 +1508,10 @@ done: compilation time. When the number of virtual operands needed to represent aliased - loads and stores grows too large (configurable with @option{--param - max-aliased-vops}), alias sets are grouped to avoid severe - compile-time slow downs and memory consumption. See group_aliases. */ + loads and stores grows too large (configurable with option --param + max-aliased-vops and --param avg-aliased-vops), alias sets are + grouped to avoid severe compile-time slow downs and memory + consumption. See compute_memory_partitions. */ static unsigned int compute_may_aliases (void) @@ -923,7 +1550,7 @@ compute_may_aliases (void) contains a mixture of pure and non-pure functions, then we need to create use-def and def-def links between these functions to avoid invalid transformations on them. */ - maybe_create_global_var (ai); + maybe_create_global_var (); /* If the program contains ref-all pointers, finalize may-alias information for them. This pass needs to be run after call-clobbering information @@ -934,30 +1561,53 @@ compute_may_aliases (void) /* Compute memory partitions for every memory variable. */ compute_memory_partitions (); - /* Debugging dumps. */ - if (dump_file) - { - dump_referenced_vars (dump_file); - if (dump_flags & TDF_STATS) - dump_alias_stats (dump_file); - dump_points_to_info (dump_file); - dump_alias_info (dump_file); - } - - /* Deallocate memory used by aliasing data structures. */ - delete_alias_info (ai); + /* Remove partitions with no symbols. Partitions may end up with an + empty MPT_SYMBOLS set if a previous round of alias analysis + needed to partition more symbols. Since we don't need those + partitions anymore, remove them to free up the space. */ + { + tree mpt; + unsigned i; + VEC(tree,heap) *mpt_table; + mpt_table = gimple_ssa_operands (cfun)->mpt_table; + i = 0; + while (i < VEC_length (tree, mpt_table)) + { + mpt = VEC_index (tree, mpt_table, i); + if (MPT_SYMBOLS (mpt) == NULL) + VEC_unordered_remove (tree, mpt_table, i); + else + i++; + } + } + + /* Populate all virtual operands and newly promoted register operands. */ { block_stmt_iterator bsi; basic_block bb; FOR_EACH_BB (bb) - { - for (bsi = bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi)) - { - update_stmt_if_modified (bsi_stmt (bsi)); - } - } + for (bsi = bsi_start (bb); !bsi_end_p (bsi); bsi_next (&bsi)) + update_stmt_if_modified (bsi_stmt (bsi)); } + + /* Debugging dumps. */ + if (dump_file) + { + dump_mem_ref_stats (dump_file); + dump_alias_info (dump_file); + dump_points_to_info (dump_file); + + if (dump_flags & TDF_STATS) + dump_alias_stats (dump_file); + + if (dump_flags & TDF_DETAILS) + dump_referenced_vars (dump_file); + } + + /* Deallocate memory used by aliasing data structures. */ + delete_alias_info (ai); + return 0; } @@ -975,8 +1625,10 @@ struct tree_opt_pass pass_may_alias = PROP_alias, /* properties_provided */ 0, /* properties_destroyed */ 0, /* todo_flags_start */ - TODO_dump_func | TODO_update_ssa - | TODO_ggc_collect | TODO_verify_ssa + TODO_dump_func + | TODO_update_ssa + | TODO_ggc_collect + | TODO_verify_ssa | TODO_verify_stmts, /* todo_flags_finish */ 0 /* letter */ }; @@ -1016,20 +1668,21 @@ count_ptr_derefs (tree *tp, int *walk_subtrees, void *data) /* Count the number of direct and indirect uses for pointer PTR in - statement STMT. The two counts are stored in *NUM_USES_P and - *NUM_DEREFS_P respectively. *IS_STORE_P is set to 'true' if at - least one of those dereferences is a store operation. */ + statement STMT. The number of direct uses is stored in + *NUM_USES_P. Indirect references are counted separately depending + on whether they are store or load operations. The counts are + stored in *NUM_STORES_P and *NUM_LOADS_P. */ void count_uses_and_derefs (tree ptr, tree stmt, unsigned *num_uses_p, - unsigned *num_derefs_p, bool *is_store) + unsigned *num_loads_p, unsigned *num_stores_p) { ssa_op_iter i; tree use; *num_uses_p = 0; - *num_derefs_p = 0; - *is_store = false; + *num_loads_p = 0; + *num_stores_p = 0; /* Find out the total number of uses of PTR in STMT. */ FOR_EACH_SSA_TREE_OPERAND (use, stmt, i, SSA_OP_USE) @@ -1073,31 +1726,79 @@ count_uses_and_derefs (tree ptr, tree stmt, unsigned *num_uses_p, rhs = stmt; } - if (lhs && (TREE_CODE (lhs) == TREE_LIST - || EXPR_P (lhs) || GIMPLE_STMT_P (lhs))) + if (lhs + && (TREE_CODE (lhs) == TREE_LIST + || EXPR_P (lhs) + || GIMPLE_STMT_P (lhs))) { struct count_ptr_d count; count.ptr = ptr; count.count = 0; walk_tree (&lhs, count_ptr_derefs, &count, NULL); - *is_store = true; - *num_derefs_p = count.count; + *num_stores_p = count.count; } - if (rhs && (TREE_CODE (rhs) == TREE_LIST - || EXPR_P (rhs) || GIMPLE_STMT_P (rhs))) + if (rhs + && (TREE_CODE (rhs) == TREE_LIST + || EXPR_P (rhs) + || GIMPLE_STMT_P (rhs))) { struct count_ptr_d count; count.ptr = ptr; count.count = 0; walk_tree (&rhs, count_ptr_derefs, &count, NULL); - *num_derefs_p += count.count; + *num_loads_p = count.count; } } - gcc_assert (*num_uses_p >= *num_derefs_p); + gcc_assert (*num_uses_p >= *num_loads_p + *num_stores_p); } + +/* Helper for delete_mem_ref_stats. Free all the slots in the + mem_sym_stats map. */ + +static bool +delete_mem_sym_stats (void *key ATTRIBUTE_UNUSED, void **value, + void *data ATTRIBUTE_UNUSED) +{ + XDELETE (*value); + *value = NULL; + return false; +} + + +/* Remove memory references stats for function FN. */ + +void +delete_mem_ref_stats (struct function *fn) +{ + if (gimple_mem_ref_stats (fn)->mem_sym_stats) + { + pointer_map_traverse (gimple_mem_ref_stats (fn)->mem_sym_stats, + delete_mem_sym_stats, NULL); + pointer_map_destroy (gimple_mem_ref_stats (fn)->mem_sym_stats); + } + + gimple_mem_ref_stats (fn)->mem_sym_stats = NULL; +} + + +/* Initialize memory reference stats. */ + +static void +init_mem_ref_stats (void) +{ + struct mem_ref_stats_d *mem_ref_stats = gimple_mem_ref_stats (cfun); + + if (mem_ref_stats->mem_sym_stats) + delete_mem_ref_stats (cfun); + + memset (mem_ref_stats, 0, sizeof (struct mem_ref_stats_d)); + mem_ref_stats->mem_sym_stats = pointer_map_create (); +} + + /* Initialize the data structures used for alias analysis. */ static struct alias_info * @@ -1115,6 +1816,9 @@ init_alias_info (void) ai->dereferenced_ptrs_store = pointer_set_create (); ai->dereferenced_ptrs_load = pointer_set_create (); + /* Clear out all memory reference stats. */ + init_mem_ref_stats (); + /* If aliases have been computed before, clear existing information. */ if (gimple_aliases_computed_p (cfun)) { @@ -1130,21 +1834,29 @@ init_alias_info (void) /* Clear flow-insensitive alias information from each symbol. */ FOR_EACH_REFERENCED_VAR (var, rvi) { + if (is_gimple_reg (var)) + continue; + if (MTAG_P (var)) MTAG_ALIASES (var) = NULL; + /* Memory partition information will be computed from scratch. */ + if (TREE_CODE (var) == MEMORY_PARTITION_TAG) + MPT_SYMBOLS (var) = NULL; + /* Since we are about to re-discover call-clobbered variables, clear the call-clobbered flag. Variables that are intrinsically call-clobbered (globals, local statics, etc) will not be marked by the aliasing code, so we can't remove them from CALL_CLOBBERED_VARS. - NB: STRUCT_FIELDS are still call clobbered if they are for - a global variable, so we *don't* clear their call clobberedness - just because they are tags, though we will clear it if they - aren't for global variables. */ + NB: STRUCT_FIELDS are still call clobbered if they are + for a global variable, so we *don't* clear their call + clobberedness just because they are tags, though we will + clear it if they aren't for global variables. */ if (TREE_CODE (var) == NAME_MEMORY_TAG || TREE_CODE (var) == SYMBOL_MEMORY_TAG + || TREE_CODE (var) == MEMORY_PARTITION_TAG || !is_global_var (var)) clear_call_clobbered (var); } @@ -1220,8 +1932,10 @@ delete_alias_info (struct alias_info *ai) delete_points_to_sets (); } + /* Used for hashing to identify pointer infos with identical pt_vars bitmaps. */ + static int eq_ptr_info (const void *p1, const void *p2) { @@ -1237,6 +1951,7 @@ ptr_info_hash (const void *p) return bitmap_hash (n->pt_vars); } + /* Create name tags for all the pointers that have been dereferenced. We only create a name tag for a pointer P if P is found to point to a set of variables (so that we can alias them to *P) or if it is @@ -1288,6 +2003,7 @@ create_name_tags (void) return; ptr_hash = htab_create (10, ptr_info_hash, eq_ptr_info, NULL); + /* Now go through the pointers with pt_vars, and find a name tag with the same pt_vars as this pointer, or create one if one doesn't exist. */ @@ -1308,7 +2024,6 @@ create_name_tags (void) problems if they both had different name tags because they would have different SSA version numbers (which would force us to take the name tags in and out of SSA). */ - slot = (struct ptr_info_def **) htab_find_slot (ptr_hash, pi, INSERT); if (*slot) pi->name_mem_tag = (*slot)->name_mem_tag; @@ -1334,12 +2049,14 @@ create_name_tags (void) /* Mark the new name tag for renaming. */ mark_sym_for_renaming (pi->name_mem_tag); } + htab_delete (ptr_hash); VEC_free (tree, heap, with_ptvars); } -/* Union the alias set SET into the may-aliases for TAG */ + +/* Union the alias set SET into the may-aliases for TAG. */ static void union_alias_set_into (tree tag, bitmap set) @@ -1784,11 +2501,13 @@ setup_pointers_and_addressables (struct alias_info *ai) to represent all possible global memory referenced by the callee. */ static void -maybe_create_global_var (struct alias_info *ai) +maybe_create_global_var (void) { /* No need to create it, if we have one already. */ if (gimple_global_var (cfun) == NULL_TREE) { + struct mem_ref_stats_d *stats = gimple_mem_ref_stats (cfun); + /* Create .GLOBAL_VAR if there are no call-clobbered variables and the program contains a mixture of pure/const and regular function calls. This is to avoid the problem @@ -1813,9 +2532,9 @@ maybe_create_global_var (struct alias_info *ai) program we create .GLOBAL_VAR to avoid missing these relations. */ if (bitmap_count_bits (gimple_call_clobbered_vars (cfun)) == 0 - && ai->num_calls_found > 0 - && ai->num_pure_const_calls_found > 0 - && ai->num_calls_found > ai->num_pure_const_calls_found) + && stats->num_call_sites > 0 + && stats->num_pure_const_call_sites > 0 + && stats->num_call_sites > stats->num_pure_const_call_sites) create_global_var (); } } @@ -1956,7 +2675,6 @@ may_alias_p (tree ptr, HOST_WIDE_INT mem_alias_set, static void add_may_alias (tree var, tree alias) { - /* Don't allow self-referential aliases. */ gcc_assert (var != alias); @@ -2285,6 +3003,10 @@ dump_alias_info (FILE *file) referenced_var_iterator rvi; tree var; + fprintf (file, "\nAlias information for %s\n\n", funcname); + + dump_memory_partitions (file); + fprintf (file, "\nFlow-insensitive alias information for %s\n\n", funcname); fprintf (file, "Aliased symbols\n\n"); @@ -2335,8 +3057,6 @@ dump_alias_info (FILE *file) dump_variable (file, var); } - dump_memory_partitions (file); - fprintf (file, "\n"); } @@ -2389,7 +3109,9 @@ dump_points_to_info_for (FILE *file, tree ptr) } if (pi->is_dereferenced) - fprintf (file, ", is dereferenced"); + fprintf (file, ", is dereferenced (R=%ld, W=%ld)", + get_mem_sym_stats_for (ptr)->num_direct_reads, + get_mem_sym_stats_for (ptr)->num_direct_writes); if (pi->value_escapes_p) fprintf (file, ", its value escapes"); @@ -2466,7 +3188,8 @@ dump_points_to_info (FILE *file) tree stmt = bsi_stmt (si); tree def; FOR_EACH_SSA_TREE_OPERAND (def, stmt, iter, SSA_OP_DEF) - if (POINTER_TYPE_P (TREE_TYPE (def))) + if (TREE_CODE (def) == SSA_NAME + && POINTER_TYPE_P (TREE_TYPE (def))) dump_points_to_info_for (file, def); } } diff --git a/gcc/tree-ssa-copy.c b/gcc/tree-ssa-copy.c index 0a17a5948dd..0bf85e5616b 100644 --- a/gcc/tree-ssa-copy.c +++ b/gcc/tree-ssa-copy.c @@ -69,17 +69,17 @@ may_propagate_copy (tree dest, tree orig) && TREE_CODE (SSA_NAME_VAR (dest)) == MEMORY_PARTITION_TAG) return (TREE_CODE (orig) == SSA_NAME && !is_gimple_reg (orig) - && (bitmap_bit_p (MPT_SYMBOLS (SSA_NAME_VAR (dest)), - DECL_UID (SSA_NAME_VAR (orig))) - || SSA_NAME_VAR (dest) == SSA_NAME_VAR (orig))); + && (SSA_NAME_VAR (dest) == SSA_NAME_VAR (orig) + || bitmap_bit_p (MPT_SYMBOLS (SSA_NAME_VAR (dest)), + DECL_UID (SSA_NAME_VAR (orig))))); if (TREE_CODE (orig) == SSA_NAME && TREE_CODE (SSA_NAME_VAR (orig)) == MEMORY_PARTITION_TAG) return (TREE_CODE (dest) == SSA_NAME && !is_gimple_reg (dest) - && (bitmap_bit_p (MPT_SYMBOLS (SSA_NAME_VAR (orig)), - DECL_UID (SSA_NAME_VAR (dest))) - || SSA_NAME_VAR (dest) == SSA_NAME_VAR (orig))); + && (SSA_NAME_VAR (dest) == SSA_NAME_VAR (orig) + || bitmap_bit_p (MPT_SYMBOLS (SSA_NAME_VAR (orig)), + DECL_UID (SSA_NAME_VAR (dest))))); /* Do not copy between types for which we *do* need a conversion. */ if (!tree_ssa_useless_type_conversion_1 (type_d, type_o)) diff --git a/gcc/tree-ssa-operands.c b/gcc/tree-ssa-operands.c index 26f4963ab5b..28464f939fc 100644 --- a/gcc/tree-ssa-operands.c +++ b/gcc/tree-ssa-operands.c @@ -2980,89 +2980,3 @@ stmt_references_memory_p (tree stmt) return stmt_ann (stmt)->references_memory; } - - -/* Return the memory partition tag (MPT) associated with memory - symbol SYM. From a correctness standpoint, memory partitions can - be assigned in any arbitrary fashion as long as this rule is - observed: Given two memory partitions MPT.i and MPT.j, they must - not contain symbols in common. - - Memory partitions are used when putting the program into Memory-SSA - form. In particular, in Memory-SSA PHI nodes are not computed for - individual memory symbols. They are computed for memory - partitions. This reduces the amount of PHI nodes in the SSA graph - at the expense of precision (i.e., it makes unrelated stores affect - each other). - - However, it is possible to increase precision by changing this - partitioning scheme. For instance, if the partitioning scheme is - such that get_mpt_for is the identity function (that is, - get_mpt_for (s) = s), this will result in ultimate precision at the - expense of huge SSA webs. - - At the other extreme, a partitioning scheme that groups all the - symbols in the same set results in minimal SSA webs and almost - total loss of precision. */ - -tree -get_mpt_for (tree sym) -{ - tree mpt; - - /* Don't create a new tag unnecessarily. */ - mpt = memory_partition (sym); - if (mpt == NULL_TREE) - { - mpt = create_tag_raw (MEMORY_PARTITION_TAG, TREE_TYPE (sym), "MPT"); - TREE_ADDRESSABLE (mpt) = 0; - MTAG_GLOBAL (mpt) = 1; - add_referenced_var (mpt); - VEC_safe_push (tree, heap, gimple_ssa_operands (cfun)->mpt_table, mpt); - MPT_SYMBOLS (mpt) = BITMAP_ALLOC (&operands_bitmap_obstack); - set_memory_partition (sym, mpt); - } - - return mpt; -} - - -/* Dump memory partition information to FILE. */ - -void -dump_memory_partitions (FILE *file) -{ - unsigned i, npart; - unsigned long nsyms; - tree mpt; - - fprintf (file, "\nMemory partitions\n\n"); - for (i = 0, npart = 0, nsyms = 0; - VEC_iterate (tree, gimple_ssa_operands (cfun)->mpt_table, i, mpt); - i++) - { - if (mpt) - { - bitmap syms = MPT_SYMBOLS (mpt); - unsigned long n = bitmap_count_bits (syms); - - fprintf (file, "#%u: ", i); - print_generic_expr (file, mpt, 0); - fprintf (file, ": %lu elements: ", n); - dump_decl_set (file, syms); - npart++; - nsyms += n; - } - } - - fprintf (file, "\n%u memory partitions holding %lu symbols\n", npart, nsyms); -} - - -/* Dump memory partition information to stderr. */ - -void -debug_memory_partitions (void) -{ - dump_memory_partitions (stderr); -} diff --git a/gcc/tree-ssa-operands.h b/gcc/tree-ssa-operands.h index 363b8586699..09b3b7bfabc 100644 --- a/gcc/tree-ssa-operands.h +++ b/gcc/tree-ssa-operands.h @@ -357,8 +357,4 @@ typedef struct ssa_operand_iterator_d /* This macro counts the number of operands in STMT matching FLAGS. */ #define NUM_SSA_OPERANDS(STMT, FLAGS) num_ssa_operands (STMT, FLAGS) -extern tree get_mpt_for (tree); -extern void dump_memory_partitions (FILE *); -extern void debug_memory_partitions (void); - #endif /* GCC_TREE_SSA_OPERANDS_H */ diff --git a/gcc/tree-ssa-structalias.c b/gcc/tree-ssa-structalias.c index b29619ec205..865770ad8a8 100644 --- a/gcc/tree-ssa-structalias.c +++ b/gcc/tree-ssa-structalias.c @@ -3011,15 +3011,21 @@ update_alias_info (tree stmt, struct alias_info *ai) bitmap addr_taken; use_operand_p use_p; ssa_op_iter iter; + bool stmt_dereferences_ptr_p; enum escape_type stmt_escape_type = is_escape_site (stmt); + struct mem_ref_stats_d *mem_ref_stats = gimple_mem_ref_stats (cfun); + + stmt_dereferences_ptr_p = false; if (stmt_escape_type == ESCAPE_TO_CALL || stmt_escape_type == ESCAPE_TO_PURE_CONST) { - ai->num_calls_found++; + mem_ref_stats->num_call_sites++; if (stmt_escape_type == ESCAPE_TO_PURE_CONST) - ai->num_pure_const_calls_found++; + mem_ref_stats->num_pure_const_call_sites++; } + else if (stmt_escape_type == ESCAPE_TO_ASM) + mem_ref_stats->num_asm_sites++; /* Mark all the variables whose address are taken by the statement. */ addr_taken = addresses_taken (stmt); @@ -3043,17 +3049,15 @@ update_alias_info (tree stmt, struct alias_info *ai) } } - /* Process each operand use. If an operand may be aliased, keep - track of how many times it's being used. For pointers, determine - whether they are dereferenced by the statement, or whether their - value escapes, etc. */ + /* Process each operand use. For pointers, determine whether they + are dereferenced by the statement, or whether their value + escapes, etc. */ FOR_EACH_PHI_OR_STMT_USE (use_p, stmt, iter, SSA_OP_USE) { tree op, var; var_ann_t v_ann; struct ptr_info_def *pi; - bool is_store, is_potential_deref; - unsigned num_uses, num_derefs; + unsigned num_uses, num_loads, num_stores; op = USE_FROM_PTR (use_p); @@ -3073,12 +3077,11 @@ update_alias_info (tree stmt, struct alias_info *ai) so that they can be treated like regular statements? Currently, they are treated as second-class statements. */ - add_to_addressable_set (TREE_OPERAND (op, 0), - &addressable_vars); + add_to_addressable_set (TREE_OPERAND (op, 0), &addressable_vars); continue; } - /* Ignore constants. */ + /* Ignore constants (they may occur in PHI node arguments). */ if (TREE_CODE (op) != SSA_NAME) continue; @@ -3109,7 +3112,7 @@ update_alias_info (tree stmt, struct alias_info *ai) /* Determine whether OP is a dereferenced pointer, and if STMT is an escape point, whether OP escapes. */ - count_uses_and_derefs (op, stmt, &num_uses, &num_derefs, &is_store); + count_uses_and_derefs (op, stmt, &num_uses, &num_loads, &num_stores); /* Handle a corner case involving address expressions of the form '&PTR->FLD'. The problem with these expressions is that @@ -3132,7 +3135,6 @@ update_alias_info (tree stmt, struct alias_info *ai) are not GIMPLE invariants), they can only appear on the RHS of an assignment and their base address is always an INDIRECT_REF expression. */ - is_potential_deref = false; if (TREE_CODE (stmt) == GIMPLE_MODIFY_STMT && TREE_CODE (GIMPLE_STMT_OPERAND (stmt, 1)) == ADDR_EXPR && !is_gimple_val (GIMPLE_STMT_OPERAND (stmt, 1))) @@ -3143,10 +3145,10 @@ update_alias_info (tree stmt, struct alias_info *ai) tree base = get_base_address (TREE_OPERAND (rhs, 0)); if (TREE_CODE (base) == INDIRECT_REF && TREE_OPERAND (base, 0) == op) - is_potential_deref = true; + num_loads++; } - if (num_derefs > 0 || is_potential_deref) + if (num_loads + num_stores > 0) { /* Mark OP as dereferenced. In a subsequent pass, dereferenced pointers that point to a set of @@ -3157,13 +3159,20 @@ update_alias_info (tree stmt, struct alias_info *ai) /* If this is a store operation, mark OP as being dereferenced to store, otherwise mark it as being dereferenced to load. */ - if (is_store) + if (num_stores > 0) pointer_set_insert (ai->dereferenced_ptrs_store, var); else pointer_set_insert (ai->dereferenced_ptrs_load, var); + + /* Update the frequency estimate for all the dereferences of + pointer OP. */ + update_mem_sym_stats_from_stmt (op, stmt, num_loads, num_stores); + + /* Indicate that STMT contains pointer dereferences. */ + stmt_dereferences_ptr_p = true; } - if (stmt_escape_type != NO_ESCAPE && num_derefs < num_uses) + if (stmt_escape_type != NO_ESCAPE && num_loads + num_stores < num_uses) { /* If STMT is an escape point and STMT contains at least one direct use of OP, then the value of OP @@ -3188,13 +3197,55 @@ update_alias_info (tree stmt, struct alias_info *ai) return; /* Mark stored variables in STMT as being written to and update the - reference counter for potentially aliased symbols in STMT. */ - if (stmt_references_memory_p (stmt) && STORED_SYMS (stmt)) + memory reference stats for all memory symbols referenced by STMT. */ + if (stmt_references_memory_p (stmt)) { unsigned i; bitmap_iterator bi; - EXECUTE_IF_SET_IN_BITMAP (STORED_SYMS (stmt), 0, i, bi) - pointer_set_insert (ai->written_vars, referenced_var (i)); + + mem_ref_stats->num_mem_stmts++; + + /* Notice that we only update memory reference stats for symbols + loaded and stored by the statement if the statement does not + contain pointer dereferences and it is not a call/asm site. + This is to avoid double accounting problems when creating + memory partitions. After computing points-to information, + pointer dereference statistics are used to update the + reference stats of the pointed-to variables, so here we + should only update direct references to symbols. + + Indirect references are not updated here for two reasons: (1) + The first time we compute alias information, the sets + LOADED/STORED are empty for pointer dereferences, (2) After + partitioning, LOADED/STORED may have references to + partitions, not the original pointed-to variables. So, if we + always counted LOADED/STORED here and during partitioning, we + would count many symbols more than once. + + This does cause some imprecision when a statement has a + combination of direct symbol references and pointer + dereferences (e.g., MEMORY_VAR = *PTR) or if a call site has + memory symbols in its argument list, but these cases do not + occur so frequently as to constitue a serious problem. */ + if (STORED_SYMS (stmt)) + EXECUTE_IF_SET_IN_BITMAP (STORED_SYMS (stmt), 0, i, bi) + { + tree sym = referenced_var (i); + pointer_set_insert (ai->written_vars, sym); + if (!stmt_dereferences_ptr_p + && stmt_escape_type != ESCAPE_TO_CALL + && stmt_escape_type != ESCAPE_TO_PURE_CONST + && stmt_escape_type != ESCAPE_TO_ASM) + update_mem_sym_stats_from_stmt (sym, stmt, 0, 1); + } + + if (!stmt_dereferences_ptr_p + && LOADED_SYMS (stmt) + && stmt_escape_type != ESCAPE_TO_CALL + && stmt_escape_type != ESCAPE_TO_PURE_CONST + && stmt_escape_type != ESCAPE_TO_ASM) + EXECUTE_IF_SET_IN_BITMAP (LOADED_SYMS (stmt), 0, i, bi) + update_mem_sym_stats_from_stmt (referenced_var (i), stmt, 1, 0); } } @@ -4108,11 +4159,11 @@ intra_create_variable_infos (void) if (!could_have_pointers (t)) continue; - /* With flag_argument_noalias greater than two means that the incoming - argument cannot alias anything except for itself so create a HEAP - variable. */ - if (POINTER_TYPE_P (TREE_TYPE (t)) - && flag_argument_noalias > 2) + /* If flag_argument_noalias is set, then function pointer + arguments are guaranteed not to point to each other. In that + case, create an artificial variable PARM_NOALIAS and the + constraint ARG = &PARM_NOALIAS. */ + if (POINTER_TYPE_P (TREE_TYPE (t)) && flag_argument_noalias > 0) { varinfo_t vi; tree heapvar = heapvar_lookup (t); @@ -4123,14 +4174,26 @@ intra_create_variable_infos (void) if (heapvar == NULL_TREE) { + var_ann_t ann; heapvar = create_tmp_var_raw (TREE_TYPE (TREE_TYPE (t)), "PARM_NOALIAS"); - get_var_ann (heapvar)->is_heapvar = 1; DECL_EXTERNAL (heapvar) = 1; if (gimple_referenced_vars (cfun)) add_referenced_var (heapvar); + heapvar_insert (t, heapvar); + + ann = get_var_ann (heapvar); + if (flag_argument_noalias == 1) + ann->noalias_state = NO_ALIAS; + else if (flag_argument_noalias == 2) + ann->noalias_state = NO_ALIAS_GLOBAL; + else if (flag_argument_noalias == 3) + ann->noalias_state = NO_ALIAS_ANYTHING; + else + gcc_unreachable (); } + vi = get_vi_for_tree (heapvar); vi->is_artificial_var = 1; vi->is_heap_var = 1; diff --git a/gcc/tree-ssa-structalias.h b/gcc/tree-ssa-structalias.h index 20b334cd86e..0f09be2cd81 100644 --- a/gcc/tree-ssa-structalias.h +++ b/gcc/tree-ssa-structalias.h @@ -49,12 +49,6 @@ struct alias_info struct alias_map_d **pointers; size_t num_pointers; - /* Number of function calls found in the program. */ - size_t num_calls_found; - - /* Number of const/pure function calls found in the program. */ - size_t num_pure_const_calls_found; - /* Variables that have been written to directly (i.e., not through a pointer dereference). */ struct pointer_set_t *written_vars; @@ -71,6 +65,7 @@ struct alias_info /* In tree-ssa-alias.c. */ enum escape_type is_escape_site (tree); +void update_mem_sym_stats_from_stmt (tree, tree, long, long); /* In tree-ssa-structalias.c. */ extern void compute_points_to_sets (struct alias_info *); diff --git a/gcc/tree-ssa.c b/gcc/tree-ssa.c index e7cd4a17c2e..1eb407d52b5 100644 --- a/gcc/tree-ssa.c +++ b/gcc/tree-ssa.c @@ -538,6 +538,51 @@ verify_call_clobbering (void) internal_error ("verify_call_clobbering failed"); } + +/* Verify invariants in memory partitions. */ + +static void +verify_memory_partitions (void) +{ + unsigned i; + tree mpt; + VEC(tree,heap) *mpt_table = gimple_ssa_operands (cfun)->mpt_table; + struct pointer_set_t *partitioned_syms = pointer_set_create (); + + for (i = 0; VEC_iterate (tree, mpt_table, i, mpt); i++) + { + unsigned j; + bitmap_iterator bj; + + if (MPT_SYMBOLS (mpt) == NULL) + { + error ("Memory partitions should have at least one symbol"); + debug_variable (mpt); + goto err; + } + + EXECUTE_IF_SET_IN_BITMAP (MPT_SYMBOLS (mpt), 0, j, bj) + { + tree var = referenced_var (j); + if (pointer_set_insert (partitioned_syms, var)) + { + error ("Partitioned symbols should belong to exactly one " + "partition"); + debug_variable (var); + goto err; + } + } + } + + pointer_set_destroy (partitioned_syms); + + return; + +err: + internal_error ("verify_memory_partitions failed"); +} + + /* Verify the consistency of aliasing information. */ static void @@ -546,6 +591,7 @@ verify_alias_info (void) verify_flow_sensitive_alias_info (); verify_call_clobbering (); verify_flow_insensitive_alias_info (); + verify_memory_partitions (); } @@ -835,6 +881,7 @@ delete_tree_ssa (void) gcc_assert (!need_ssa_update_p ()); } cfun->gimple_df->aliases_computed_p = false; + delete_mem_ref_stats (cfun); cfun->gimple_df = NULL; } diff --git a/gcc/tree-vrp.c b/gcc/tree-vrp.c index dbb97de887a..102d2be1a9d 100644 --- a/gcc/tree-vrp.c +++ b/gcc/tree-vrp.c @@ -3121,11 +3121,10 @@ infer_value_range (tree stmt, tree op, enum tree_code *comp_code_p, tree *val_p) non-NULL if -fdelete-null-pointer-checks is enabled. */ if (flag_delete_null_pointer_checks && POINTER_TYPE_P (TREE_TYPE (op))) { - bool is_store; - unsigned num_uses, num_derefs; + unsigned num_uses, num_loads, num_stores; - count_uses_and_derefs (op, stmt, &num_uses, &num_derefs, &is_store); - if (num_derefs > 0) + count_uses_and_derefs (op, stmt, &num_uses, &num_loads, &num_stores); + if (num_loads + num_stores > 0) { *val_p = build_int_cst (TREE_TYPE (op), 0); *comp_code_p = NE_EXPR;