On Mon, Mar 17, 2025 at 11:54 PM Andrew Pinski <quic_apin...@quicinc.com> wrote:
>
> While debugging a failure, I noticed that SCC copy didn't print
> out what it was doing, e.g. replacing name1 with name 2.
> This adds that dump.
>
> Bootstrapped and tested on x86_64-linux-gnu.
OK.
> gcc/ChangeLog:
>
> * gimple-ssa-sccopy.cc (scc_copy_prop::replace_scc_by_value): Dump
> what is being replaced with what.
>
> Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com>
> ---
> gcc/gimple-ssa-sccopy.cc | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/gcc/gimple-ssa-sccopy.cc b/gcc/gimple-ssa-sccopy.cc
> index 7ffb5718ab6..298feb05571 100644
> --- a/gcc/gimple-ssa-sccopy.cc
> +++ b/gcc/gimple-ssa-sccopy.cc
> @@ -38,6 +38,7 @@ along with GCC; see the file COPYING3. If not see
> #include "builtins.h"
> #include "tree-ssa-dce.h"
> #include "fold-const.h"
> +#include "tree-pretty-print.h"
>
> /* Strongly connected copy propagation pass.
>
> @@ -485,6 +486,15 @@ scc_copy_prop::replace_scc_by_value (vec<gimple *> scc,
> tree val)
> for (gimple *stmt : scc)
> {
> tree name = gimple_get_lhs (stmt);
> + if (dump_file && (dump_flags & TDF_DETAILS))
> + {
> + fprintf (dump_file, "Replacing ");
> + print_generic_expr (dump_file, name);
> + fprintf (dump_file, " with ");
> + print_generic_expr (dump_file, val);
> + fprintf (dump_file, "\n");
> +
> + }
> replace_uses_by (name, val);
> bitmap_set_bit (dead_stmts, SSA_NAME_VERSION (name));
> }
> --
> 2.43.0
>
