diff mbox

[ping] Change behavior of -fsched-verbose option

Message ID 563B0F24.7010008@samsung.com
State New
Headers show

Commit Message

Nikolai Bozhenov Nov. 5, 2015, 8:11 a.m. UTC
Please commit if it is OK for trunk.

Thanks,
Nikolai

On 10/22/2015 06:38 PM, Nikolai Bozhenov wrote:
> Hi!
>
> Currently -fsched-verbose option redirects debugging dumps to stderr
> if there is no dump_file for the current pass. It would be fine if
> there were the only scheduling pass. But for example for AArch64
> there are 3 scheduling passes in the default pipeline: sched1,
> fusion and sched2. So, when passing options like
> -fsched-verbose=7 -fdump-rtl-sched1 to GCC I get a neat dump for
> sched1 in the file and a mess of fusion/sched2 logs in the console.
>
> In fact, currently there's no way to tell GCC that I want extremely
> verbose logs for sched1 and I want no logs for all other passes.
> Especially to the console.
>
> I suggest disabling such redirection in the scheduler and omitting
> debugging output for passes without dump_file. I believe a better
> way to redirect printing to the stderr is to use options like
> -fdump-rtl-sched=stderr. The attached patch implements the
> suggestion.
>
> Anyway, the old behavior may be reproduced with options
> -fsched-verbose=7 -fdump-rtl-sched1 
> -fdump-rtl-{sched_fusion,sched2}=stderr
> if it is really necessary.
>
> The patch has been bootstrapped and regtested on x86_64.
>
> Thanks,
> Nikolai

Comments

Bernd Schmidt Nov. 5, 2015, 9:26 a.m. UTC | #1
On 11/05/2015 09:11 AM, Nikolai Bozhenov wrote:
> Please commit if it is OK for trunk.

Hmm, do you have a copyright assignment on file?


Bernd
Nikolai Bozhenov Nov. 5, 2015, 10:20 a.m. UTC | #2
On 11/05/2015 12:26 PM, Bernd Schmidt wrote:
> On 11/05/2015 09:11 AM, Nikolai Bozhenov wrote:
>> Please commit if it is OK for trunk.
>
> Hmm, do you have a copyright assignment on file?
>

Yes I do have a copyright assignment for all past and future changes
to GCC (RT:828836).

Thanks,
Nikolai
diff mbox

Patch

2015-10-22  Nikolai Bozhenov  <n.bozhenov@samsung.com>

	* haifa-sched.c (setup_sched_dump): Don't redirect output to stderr.
	* common.opt (-fsched-verbose): Set default value to 1.
	* invoke.texi (-fsched-verbose): Update the option's description.

diff --git a/gcc/common.opt b/gcc/common.opt
index 961a1b6..757ce85 100644
--- a/gcc/common.opt
+++ b/gcc/common.opt
@@ -1967,17 +1967,17 @@  fsched-spec-load
 Common Report Var(flag_schedule_speculative_load) Optimization
 Allow speculative motion of some loads.
 
 fsched-spec-load-dangerous
 Common Report Var(flag_schedule_speculative_load_dangerous) Optimization
 Allow speculative motion of more loads.
 
 fsched-verbose=
-Common RejectNegative Joined UInteger Var(sched_verbose_param)
+Common RejectNegative Joined UInteger Var(sched_verbose_param) Init(1)
 -fsched-verbose=<number>	Set the verbosity level of the scheduler.
 
 fsched2-use-superblocks
 Common Report Var(flag_sched2_use_superblocks) Optimization
 If scheduling post reload, do superblock scheduling.
 
 fsched2-use-traces
 Common Ignore
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 4fc7d88..11b8697 100644
--- a/gcc/doc/invoke.texi
+++ b/gcc/doc/invoke.texi
@@ -7415,22 +7415,17 @@  place unique stamps in coverage data files and the object files that
 produce them.  You can use the @option{-frandom-seed} option to produce
 reproducibly identical object files.
 
 The @var{number} should be different for every file you compile.
 
 @item -fsched-verbose=@var{n}
 @opindex fsched-verbose
 On targets that use instruction scheduling, this option controls the
-amount of debugging output the scheduler prints.  This information is
-written to standard error, unless @option{-fdump-rtl-sched1} or
-@option{-fdump-rtl-sched2} is specified, in which case it is output
-to the usual dump listing file, @file{.sched1} or @file{.sched2}
-respectively.  However for @var{n} greater than nine, the output is
-always printed to standard error.
+amount of debugging output the scheduler prints to the dump files.
 
 For @var{n} greater than zero, @option{-fsched-verbose} outputs the
 same information as @option{-fdump-rtl-sched1} and @option{-fdump-rtl-sched2}.
 For @var{n} greater than one, it also output basic block probabilities,
 detailed ready list information and unit/insn info.  For @var{n} greater
 than two, it includes RTL at abort point, control-flow and regions info.
 And for @var{n} over four, @option{-fsched-verbose} also includes
 dependence info.
diff --git a/gcc/haifa-sched.c b/gcc/haifa-sched.c
index caadc11..835648b 100644
--- a/gcc/haifa-sched.c
+++ b/gcc/haifa-sched.c
@@ -198,27 +198,24 @@  static int modulo_iter0_max_uid;
 static int modulo_backtracks_left;
 
 /* The stage in which the last insn from the original loop was
    scheduled.  */
 static int modulo_last_stage;
 
 /* sched-verbose controls the amount of debugging output the
    scheduler prints.  It is controlled by -fsched-verbose=N:
-   N>0 and no -DSR : the output is directed to stderr.
-   N>=10 will direct the printouts to stderr (regardless of -dSR).
-   N=1: same as -dSR.
+   N=0: no debugging output.
+   N=1: default value.
    N=2: bb's probabilities, detailed ready list info, unit/insn info.
    N=3: rtl at abort point, control-flow, regions info.
    N=5: dependences info.  */
-
 int sched_verbose = 0;
 
-/* Debugging file.  All printouts are sent to dump, which is always set,
-   either to stderr, or to the dump listing file (-dRS).  */
+/* Debugging file.  All printouts are sent to dump. */
 FILE *sched_dump = 0;
 
 /* This is a placeholder for the scheduler parameters common
    to all schedulers.  */
 struct common_sched_info_def *common_sched_info;
 
 #define INSN_TICK(INSN)	(HID (INSN)->tick)
 #define INSN_EXACT_TICK(INSN) (HID (INSN)->exact_tick)
@@ -7214,27 +7211,24 @@  set_priorities (rtx_insn *head, rtx_insn *tail)
 				      INSN_PRIORITY (insn));
     }
 
   current_sched_info->sched_max_insns_priority = sched_max_insns_priority;
 
   return n_insn;
 }
 
-/* Set dump and sched_verbose for the desired debugging output.  If no
-   dump-file was specified, but -fsched-verbose=N (any N), print to stderr.
-   For -fsched-verbose=N, N>=10, print everything to stderr.  */
+/* Set sched_dump and sched_verbose for the desired debugging output. */
 void
 setup_sched_dump (void)
 {
   sched_verbose = sched_verbose_param;
-  if (sched_verbose_param == 0 && dump_file)
-    sched_verbose = 1;
-  sched_dump = ((sched_verbose_param >= 10 || !dump_file)
-		? stderr : dump_file);
+  sched_dump = dump_file;
+  if (!dump_file)
+    sched_verbose = 0;
 }
 
 /* Allocate data for register pressure sensitive scheduling.  */
 static void
 alloc_global_sched_pressure_data (void)
 {
   if (sched_pressure != SCHED_PRESSURE_NONE)
     {