diff options
| author | Jan Hubicka <jh@suse.cz> | 2020-10-22 17:32:32 +0200 |
|---|---|---|
| committer | Jan Hubicka <jh@suse.cz> | 2020-10-22 17:32:32 +0200 |
| commit | 0e590b68fa3743656f40aee8374b788b108350c7 (patch) | |
| tree | 1183a176f4543e9e141d2a2657e1212e6f1dc7af /gcc/cgraph.c | |
| parent | c26d7df103197e52dcd6edbb9a7f58eafdd6c715 (diff) | |
| download | gcc-0e590b68fa3743656f40aee8374b788b108350c7.zip gcc-0e590b68fa3743656f40aee8374b788b108350c7.tar.gz gcc-0e590b68fa3743656f40aee8374b788b108350c7.tar.bz2 | |
Materialize clones on demand
this patch removes the pass to materialize all clones and instead this
is now done on demand. The motivation is to reduce lifetime of function
bodies in ltrans that should noticeably reduce memory use for highly
parallel compilations of large programs (like Martin does) or with
partitioning reduced/disabled. For cc1 with one partition the memory use
seems to go down from 4gb to cca 1.5gb (seeing from top, so this is not
particularly accurate).
gcc/ChangeLog:
2020-10-22 Jan Hubicka <hubicka@ucw.cz>
* cgraph.c (cgraph_node::get_untransformed_body): Perform lazy
clone materialization.
* cgraph.h (cgraph_node::materialize_clone): Declare.
(symbol_table::materialize_all_clones): Remove.
* cgraphclones.c (cgraph_materialize_clone): Turn to ...
(cgraph_node::materialize_clone): .. this one; move here
dumping from symbol_table::materialize_all_clones.
(symbol_table::materialize_all_clones): Remove.
* cgraphunit.c (mark_functions_to_output): Clear stmt references.
(cgraph_node::expand): Initialize bitmaps early;
do not call execute_all_ipa_transforms if there are no transforms.
* ipa-inline-transform.c (save_inline_function_body): Fix formating.
(inline_transform): Materialize all clones before function is modified.
* ipa-param-manipulation.c (ipa_param_adjustments::modify_call):
Materialize clone if needed.
* ipa.c (class pass_materialize_all_clones): Remove.
(make_pass_materialize_all_clones): Remove.
* passes.c (execute_all_ipa_transforms): Materialize all clones.
* passes.def: Remove pass_materialize_all_clones.
* tree-pass.h (make_pass_materialize_all_clones): Remove.
* tree-ssa-structalias.c (ipa_pta_execute): Clear refs.
Diffstat (limited to 'gcc/cgraph.c')
| -rw-r--r-- | gcc/cgraph.c | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/gcc/cgraph.c b/gcc/cgraph.c index 9480935..067984d 100644 --- a/gcc/cgraph.c +++ b/gcc/cgraph.c @@ -3872,16 +3872,27 @@ cgraph_node::function_or_virtual_thunk_symbol } /* When doing LTO, read cgraph_node's body from disk if it is not already - present. */ + present. Also perform any necessary clone materializations. */ bool -cgraph_node::get_untransformed_body (void) +cgraph_node::get_untransformed_body () { lto_file_decl_data *file_data; const char *data, *name; size_t len; tree decl = this->decl; + /* See if there is clone to be materialized. + (inline clones does not need materialization, but we can be seeing + an inline clone of real clone). */ + cgraph_node *p = this; + for (cgraph_node *c = clone_of; c; c = c->clone_of) + { + if (c->decl != decl) + p->materialize_clone (); + p = c; + } + /* Check if body is already there. Either we have gimple body or the function is thunk and in that case we set DECL_ARGUMENTS. */ if (DECL_ARGUMENTS (decl) || gimple_has_body_p (decl)) |