and used pp_format for formatting arg_positions by adding specifier %I (name chosen arbitrarily). Does that look OK ?
diff --git a/gcc/pretty-print.c b/gcc/pretty-print.c index a39815e..e8bd1ef 100644 --- a/gcc/pretty-print.c +++ b/gcc/pretty-print.c @@ -610,6 +610,23 @@ pp_format (pretty_printer *pp, text_info *text) (pp, *text->args_ptr, precision, unsigned, "u"); break; + case 'I': The comment just above pp_format that lists the format specifiers understood by the function should be updated. There probably (I hope) is more formal documentation of the format specifiers somewhere else that should be updated as well. That said, since this specifier formats a vec<int>, it seems that it might be useful to be able to format vectors of other elements, such as short and long. With that in mind, would adding a new V length modifier instead be a more regular way to extend the pretty printer to these types? The V length modifier would have to be accepted in conjunction with other length modifiers (h, l, etc and type specifiers (d, i, o, etc.). E.g, "%hVu" would accept a vec<unsigned short>* as an argument and format it as a sequence of decimal numbers, and "%lVx" would do the same for vec<long>* formatting them in hex. Martin