| Message ID | 20210107230758.3110298-1-slyfox@gentoo.org |
|---|---|
| State | New |
| Headers | show |
| Series | ipa-modref: avoid linebreak split in debug print | expand |
On Thu, 7 Jan 2021, Sergei Trofimovich wrote: > From: Sergei Trofimovich <siarheit@google.com> > > * ipa-modref.c (merge_call_side_effects): Fix > linebreak split by reordering two print calls. OK. Richard. > --- > gcc/ipa-modref.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/gcc/ipa-modref.c b/gcc/ipa-modref.c > index fcc676d25e4..04613201f1f 100644 > --- a/gcc/ipa-modref.c > +++ b/gcc/ipa-modref.c > @@ -835,10 +835,6 @@ merge_call_side_effects (modref_summary *cur_summary, > auto_vec <modref_parm_map, 32> parm_map; > bool changed = false; > > - if (dump_file) > - fprintf (dump_file, " - Merging side effects of %s with parm map:", > - callee_node->dump_name ()); > - > /* We can not safely optimize based on summary of callee if it does > not always bind to current def: it is possible that memory load > was optimized out earlier which may not happen in the interposed > @@ -850,6 +846,10 @@ merge_call_side_effects (modref_summary *cur_summary, > cur_summary->loads->collapse (); > } > > + if (dump_file) > + fprintf (dump_file, " - Merging side effects of %s with parm map:", > + callee_node->dump_name ()); > + > parm_map.safe_grow_cleared (gimple_call_num_args (stmt), true); > for (unsigned i = 0; i < gimple_call_num_args (stmt); i++) > { >
diff --git a/gcc/ipa-modref.c b/gcc/ipa-modref.c index fcc676d25e4..04613201f1f 100644 --- a/gcc/ipa-modref.c +++ b/gcc/ipa-modref.c @@ -835,10 +835,6 @@ merge_call_side_effects (modref_summary *cur_summary, auto_vec <modref_parm_map, 32> parm_map; bool changed = false; - if (dump_file) - fprintf (dump_file, " - Merging side effects of %s with parm map:", - callee_node->dump_name ()); - /* We can not safely optimize based on summary of callee if it does not always bind to current def: it is possible that memory load was optimized out earlier which may not happen in the interposed @@ -850,6 +846,10 @@ merge_call_side_effects (modref_summary *cur_summary, cur_summary->loads->collapse (); } + if (dump_file) + fprintf (dump_file, " - Merging side effects of %s with parm map:", + callee_node->dump_name ()); + parm_map.safe_grow_cleared (gimple_call_num_args (stmt), true); for (unsigned i = 0; i < gimple_call_num_args (stmt); i++) {
From: Sergei Trofimovich <siarheit@google.com> * ipa-modref.c (merge_call_side_effects): Fix linebreak split by reordering two print calls. --- gcc/ipa-modref.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)