Re: [PATCH] PR bootstrap/71471: remove selftest for pp_format (%p)

[Jeff Law]

On 06/09/2016 10:45 AM, David Malcolm wrote:

I was confused by the comment to pp_format:

  /* The following format specifiers are recognized as being client independent:
     ...
     %p: pointer
     ... */

into thinking that %p is printed in a host-independent manner, when
"client independent" means in relation to different pretty-printers e.g the
C++ pretty printer, or normal tree one etc.

The following patch removes the overzealous test case and clarifies
the comment.

OK for trunk?

I'm working on a followup that makes selftest failures easier to
track down.

gcc/ChangeLog:
        PR bootstrap/71471
        * pretty-print.c (pp_indent): Specify that %p is printed in a
        host-dependent manner.
        (test_pp_format): Remove the test for %p.

OK.
jeff