On Wed, Dec 21, 2011 at 11:32 AM, Martin Jambor <mjam...@suse.cz> wrote:
> Hi,
>
> I know it's stage three but I hope this tiny patch which affects only
> dumping is still acceptable. In SRA passes I wrote dumping so that
> when the detailed dump is not requested, it is quite brief and only
> writes the decisions, not how it reached those decisions. Of course,
> that means that debugging SRA I make heavy use of detailed dumps.
>
> The slightly annoying thing is that detailed dumps switch on
> tree-into-ssa dumps at all, but the really annoying thing is just how
> many blank lines it writes into the dump. The function body and my
> dumps are quite far apart and can look quite confusing. Yesterday I
> got fed up to the level that I wrote the patch below which cuts back
> on newlines significantly.
>
> It also adds one to the beginning of update_ssa() so that there always
> is a blank line dividing these dumps from whatever there is before
> them. Otherwise, it just removes superfluous "\n" stuff.
>
> I actually included this in a bootstrap and testsuite run,
> unsurprisingly it caused no issues. OK for trunk now?
Ok.
Richard.
> Thanks,
>
> Martin
>
>
>
> 2011-12-20 Martin Jambor <mjam...@suse.cz>
>
> * tree-into-ssa.c (rewrite_update_stmt): Do not dump extra newlines.
> (rewrite_update_enter_block): Likewise.
> (dump_update_ssa): Likewise.
> (update_ssa): Likewise but also dump a newline at the beginning.
>
> Index: src/gcc/tree-into-ssa.c
> ===================================================================
> --- src.orig/gcc/tree-into-ssa.c
> +++ src/gcc/tree-into-ssa.c
> @@ -2051,7 +2051,6 @@ rewrite_update_stmt (gimple stmt, gimple
> {
> fprintf (dump_file, "Updating SSA information for statement ");
> print_gimple_stmt (dump_file, stmt, 0, TDF_SLIM);
> - fprintf (dump_file, "\n");
> }
>
> /* Rewrite USES included in OLD_SSA_NAMES and USES whose underlying
> @@ -2199,7 +2198,7 @@ rewrite_update_enter_block (struct dom_w
> gimple_stmt_iterator gsi;
>
> if (dump_file && (dump_flags & TDF_DETAILS))
> - fprintf (dump_file, "\n\nRegistering new PHI nodes in block #%d\n\n",
> + fprintf (dump_file, "Registering new PHI nodes in block #%d\n",
> bb->index);
>
> /* Mark the unwind point for this block. */
> @@ -2848,22 +2847,21 @@ dump_update_ssa (FILE *file)
>
> if (!bitmap_empty_p (SYMS_TO_RENAME (cfun)))
> {
> - fprintf (file, "\n\nSymbols to be put in SSA form\n\n");
> + fprintf (file, "\nSymbols to be put in SSA form\n");
> dump_decl_set (file, SYMS_TO_RENAME (cfun));
> fprintf (file, "\n");
> }
>
> if (names_to_release && !bitmap_empty_p (names_to_release))
> {
> - fprintf (file, "\n\nSSA names to release after updating the SSA
> web\n\n");
> + fprintf (file, "\nSSA names to release after updating the SSA
> web\n\n");
> EXECUTE_IF_SET_IN_BITMAP (names_to_release, 0, i, bi)
> {
> print_generic_expr (file, ssa_name (i), 0);
> fprintf (file, " ");
> }
> + fprintf (file, "\n");
> }
> -
> - fprintf (file, "\n\n");
> }
>
>
> @@ -3342,6 +3340,9 @@ update_ssa (unsigned update_flags)
>
> timevar_push (TV_TREE_SSA_INCREMENTAL);
>
> + if (dump_file && (dump_flags & TDF_DETAILS))
> + fprintf (dump_file, "\nUpdating SSA:\n");
> +
> if (!update_ssa_initialized_fn)
> init_update_ssa (cfun);
> gcc_assert (update_ssa_initialized_fn == cfun);
> @@ -3506,14 +3507,14 @@ update_ssa (unsigned update_flags)
>
> dump_update_ssa (dump_file);
>
> - fprintf (dump_file, "Incremental SSA update started at block: %d\n\n",
> + fprintf (dump_file, "Incremental SSA update started at block: %d\n",
> start_bb->index);
>
> c = 0;
> EXECUTE_IF_SET_IN_BITMAP (blocks_to_update, 0, i, bi)
> c++;
> fprintf (dump_file, "Number of blocks in CFG: %d\n", last_basic_block);
> - fprintf (dump_file, "Number of blocks to update: %d (%3.0f%%)\n\n",
> + fprintf (dump_file, "Number of blocks to update: %d (%3.0f%%)\n",
> c, PERCENT (c, last_basic_block));
>
> if (dump_flags & TDF_DETAILS)
>
