This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Enhance dump_probability function.
- From: Paul Hua <paul dot hua dot gm at gmail dot com>
- To: Martin Liška <mliska at suse dot cz>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 14 Jun 2017 10:02:11 +0800
- Subject: Re: [PATCH] Enhance dump_probability function.
- Authentication-results: sourceware.org; auth=none
- References: <f776d98c-d5db-edd7-54b8-995b3eb08a19@suse.cz>
New "ERROR: (DejaGnu)" on mips64el target.
my DejaGnu version is 1.5.1.
1)
make check-gcc RUNTESTFLAGS="tree-ssa.exp=builtin-sprintf-2.c"
...
ERROR: (DejaGnu) proc "^:\\" does not exist.
The error code is TCL LOOKUP COMMAND ^:\\
The info on the error is:
invalid command name "^:\"
while executing
"::tcl_unknown ^:\\"
("uplevel" body line 1)
invoked from within
"uplevel 1 ::tcl_unknown $args"
...
2)
make check-gcc RUNTESTFLAGS="tree-ssa.exp=vrp101.c"
...
ERROR: (DejaGnu) proc "^:\\" does not exist.
The error code is TCL LOOKUP COMMAND ^:\\
The info on the error is:
invalid command name "^:\"
while executing
"::tcl_unknown ^:\\"
("uplevel" body line 1)
invoked from within
"uplevel 1 ::tcl_unknown $args"
...
I don't known how to debug this, any advice ?
Paul.
On Tue, Jun 13, 2017 at 4:14 PM, Martin Liška <mliska@suse.cz> wrote:
> Hi.
>
> This is pre-approved patch that displays edge counts in dump files:
>
> ...
> _85 = _83 + _84;
> len_86 = SQRT (_85);
> if (_85 u>= 0.0)
> goto <bb 6>; [99.00%] [count: 778568]
> else
> goto <bb 5>; [1.00%] [count: 7864]
>
> <bb 5> [0.01%] [count: 7864]:
> sqrt (_85);
> ...
>
> That makes it possible to understand why a profile mismatch happens.
> Patch can bootstrap on ppc64le-redhat-linux and survives regression tests.
>
> Martin
>
> gcc/ChangeLog:
>
> 2017-06-12 Martin Liska <mliska@suse.cz>
>
> * gimple-pretty-print.c (dump_probability): Add new argument.
> (dump_edge_probability): Dump both probability and count.
> (dump_gimple_label): Likewise.
> (dump_gimple_bb_header): Likewise.
>
> gcc/testsuite/ChangeLog:
>
> 2017-06-12 Martin Liska <mliska@suse.cz>
>
> * gcc.dg/tree-ssa/builtin-sprintf-2.c: Adjust scanned pattern.
> * gcc.dg/tree-ssa/dump-2.c: Likewise.
> * gcc.dg/tree-ssa/vrp101.c: Likewise.
> ---
> gcc/gimple-pretty-print.c | 22 ++++++++++++++--------
> gcc/testsuite/gcc.dg/tree-ssa/builtin-sprintf-2.c | 4 ++--
> gcc/testsuite/gcc.dg/tree-ssa/dump-2.c | 2 +-
> gcc/testsuite/gcc.dg/tree-ssa/vrp101.c | 2 +-
> 4 files changed, 18 insertions(+), 12 deletions(-)
>
>