Hello, sched-vis.c would print insns in a SEQUENCE as a pattern, in effect printing only GET_CODE(insn). Fixed with this patch.
Bootstrapped&tested on powerpc64-unknown-linux-gnu and sparc64-unknown-linux-gnu with some dump inspections to make sure everything looks as expected now. OK for trunk? Ciao! Steven
* sched-vis.c (print_pattern): Print SEQUENCE of insns as insns. Index: sched-vis.c =================================================================== --- sched-vis.c (revision 197610) +++ sched-vis.c (working copy) @@ -51,6 +51,9 @@ along with GCC; see the file COPYING3. If not see static bool rtl_slim_pp_initialized = false; static pretty_printer rtl_slim_pp; +/* For insns we print patterns, and for some patterns we print insns... */ +static void print_insn_with_notes (pretty_printer *, const_rtx); + /* This recognizes rtx'en classified as expressions. These are always represent some action on values or results of other expression, that may be stored in objects representing values. */ @@ -562,14 +565,32 @@ print_pattern (pretty_printer *pp, const_rtx x, in break; case SEQUENCE: { - int i; - pp_string (pp, "sequence{"); - for (i = 0; i < XVECLEN (x, 0); i++) + if (INSN_P (XVECEXP (x, 0, 0))) { - print_pattern (pp, XVECEXP (x, 0, i), verbose); - pp_character (pp, ';'); + /* Print the sequence insns indented. */ + const char * save_print_rtx_head = print_rtx_head; + char indented_print_rtx_head[32]; + + pp_newline (pp); + gcc_assert (strlen (print_rtx_head) < sizeof (indented_print_rtx_head) - 4); + snprintf (indented_print_rtx_head, + sizeof (indented_print_rtx_head), + "%s ", print_rtx_head); + print_rtx_head = indented_print_rtx_head; + for (int i = 0; i < XVECLEN (x, 0); i++) + print_insn_with_notes (pp, XVECEXP (x, 0, i)); + pp_printf (pp, "%s ", save_print_rtx_head); + print_rtx_head = save_print_rtx_head; } + else + { + for (int i = 0; i < XVECLEN (x, 0); i++) + { + print_pattern (pp, XVECEXP (x, 0, i), verbose); + pp_character (pp, ';'); + } + } pp_character (pp, '}'); } break;