diff options
author | Jan Hubicka <hubicka@ucw.cz> | 2017-05-22 12:39:46 +0200 |
---|---|---|
committer | Jan Hubicka <hubicka@gcc.gnu.org> | 2017-05-22 10:39:46 +0000 |
commit | 106eeddc29bcddb3df6c98b19d43b214dbb8d94f (patch) | |
tree | 136d3e0f4b6b8e3a24af513c94069b87a9a4d800 | |
parent | fbb1c7d4a061ae0f57d52b37ddf37933da859945 (diff) | |
download | gcc-106eeddc29bcddb3df6c98b19d43b214dbb8d94f.zip gcc-106eeddc29bcddb3df6c98b19d43b214dbb8d94f.tar.gz gcc-106eeddc29bcddb3df6c98b19d43b214dbb8d94f.tar.bz2 |
ipa-inline-analysis.c (inline_summary::reset): Do not reset self_time.
* ipa-inline-analysis.c (inline_summary::reset): Do not reset
self_time.
(dump_inline_summary): Do not print self_time.
(estimate_function_body_sizes): Do not set self_time.
(compute_inline_parameters): Likewise.
(inline_read_section, inline_write_summary): Do not stream self_time.
* ipa-inline.h (inline_summary): Drop self_time.
From-SVN: r248325
-rw-r--r-- | gcc/ChangeLog | 10 | ||||
-rw-r--r-- | gcc/ipa-inline-analysis.c | 15 | ||||
-rw-r--r-- | gcc/ipa-inline.h | 4 |
3 files changed, 16 insertions, 13 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 8d57bcf..0a3ea11 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,15 @@ 2017-05-22 Jan Hubicka <hubicka@ucw.cz> + * ipa-inline-analysis.c (inline_summary::reset): Do not reset + self_time. + (dump_inline_summary): Do not print self_time. + (estimate_function_body_sizes): Do not set self_time. + (compute_inline_parameters): Likewise. + (inline_read_section, inline_write_summary): Do not stream self_time. + * ipa-inline.h (inline_summary): Drop self_time. + +2017-05-22 Jan Hubicka <hubicka@ucw.cz> + * ipa-inline-analysis.c (account_size_time): Rename to ... (inline_summary::account_size_time): ... this one. (reset_ipa_call_summary): Turn to ... diff --git a/gcc/ipa-inline-analysis.c b/gcc/ipa-inline-analysis.c index 13f775d..1a88e59 100644 --- a/gcc/ipa-inline-analysis.c +++ b/gcc/ipa-inline-analysis.c @@ -560,7 +560,6 @@ inline_summary::reset (struct cgraph_node *node) struct cgraph_edge *e; self_size = 0; - self_time = 0; estimated_stack_size = 0; estimated_self_stack_size = 0; stack_frame_offset = 0; @@ -920,8 +919,7 @@ dump_inline_summary (FILE *f, struct cgraph_node *node) fprintf (f, " contains_cilk_spawn"); if (s->fp_expressions) fprintf (f, " fp_expression"); - fprintf (f, "\n self time: %f\n", s->self_time.to_double ()); - fprintf (f, " global time: %f\n", s->time.to_double ()); + fprintf (f, "\n global time: %f\n", s->time.to_double ()); fprintf (f, " self size: %i\n", s->self_size); fprintf (f, " global size: %i\n", s->size); fprintf (f, " min size: %i\n", s->min_size); @@ -2415,7 +2413,7 @@ estimate_function_body_sizes (struct cgraph_node *node, bool early) e->aux = NULL; } } - inline_summaries->get (node)->self_time = time; + inline_summaries->get (node)->time = time; inline_summaries->get (node)->self_size = size; nonconstant_names.release (); ipa_release_body_info (&fbi); @@ -2472,7 +2470,6 @@ compute_inline_parameters (struct cgraph_node *node, bool early) info->account_size_time (2 * INLINE_SIZE_SCALE, 0, t, t); inline_update_overall_summary (node); info->self_size = info->size; - info->self_time = info->time; /* We can not inline instrumentation clones. */ if (node->thunk.add_pointer_bounds_args) { @@ -2539,7 +2536,6 @@ compute_inline_parameters (struct cgraph_node *node, bool early) node->calls_comdat_local = (e != NULL); /* Inlining characteristics are maintained by the cgraph_mark_inline. */ - info->time = info->self_time; info->size = info->self_size; info->stack_frame_offset = 0; info->estimated_stack_size = info->estimated_self_stack_size; @@ -2548,8 +2544,7 @@ compute_inline_parameters (struct cgraph_node *node, bool early) inline_update_overall_summary but because computation happens in different order the roundoff errors result in slight changes. */ inline_update_overall_summary (node); - gcc_assert (!(info->time - info->self_time).to_int () - && info->size == info->self_size); + gcc_assert (info->size == info->self_size); } @@ -3695,7 +3690,7 @@ inline_read_section (struct lto_file_decl_data *file_data, const char *data, info->estimated_stack_size = info->estimated_self_stack_size = streamer_read_uhwi (&ib); info->size = info->self_size = streamer_read_uhwi (&ib); - info->time = info->self_time = sreal::stream_in (&ib); + info->time = sreal::stream_in (&ib); bp = streamer_read_bitpack (&ib); info->inlinable = bp_unpack_value (&bp, 1); @@ -3848,7 +3843,7 @@ inline_write_summary (void) streamer_write_uhwi (ob, lto_symtab_encoder_encode (encoder, cnode)); streamer_write_hwi (ob, info->estimated_self_stack_size); streamer_write_hwi (ob, info->self_size); - info->self_time.stream_out (ob); + info->time.stream_out (ob); bp = bitpack_create (ob->main_stream); bp_pack_value (&bp, info->inlinable, 1); bp_pack_value (&bp, info->contains_cilk_spawn, 1); diff --git a/gcc/ipa-inline.h b/gcc/ipa-inline.h index eb23533..675e975 100644 --- a/gcc/ipa-inline.h +++ b/gcc/ipa-inline.h @@ -96,8 +96,6 @@ struct GTY(()) inline_summary HOST_WIDE_INT estimated_self_stack_size; /* Size of the function body. */ int self_size; - /* Time of the function body. */ - sreal GTY((skip)) self_time; /* Minimal size increase after inlining. */ int min_size; @@ -149,7 +147,7 @@ struct GTY(()) inline_summary /* Keep all field empty so summary dumping works during its computation. This is useful for debugging. */ inline_summary () - : estimated_self_stack_size (0), self_size (0), self_time (0), min_size (0), + : estimated_self_stack_size (0), self_size (0), min_size (0), inlinable (false), contains_cilk_spawn (false), single_caller (false), fp_expressions (false), estimated_stack_size (false), stack_frame_offset (false), time (0), size (0), conds (NULL), |