On Fri, 25 Oct 2024, Jakub Jelinek wrote:

> On Tue, Oct 15, 2024 at 11:47:45AM +0200, Jakub Jelinek wrote:
> > The following patch adds selftests to genmatch to verify the new printing
> > routine there.
> > So that I can rely on HAVE_DECL_FMEMOPEN (host test), the tests are done
> > solely in stage2+ where we link the host libcpp etc. to genmatch.
> > The tests have been adjusted from pretty-print.cc (test_pp_format),
> > and I've added to that function two new tests because I've noticed nothing
> > was testing the %M$.*N$s etc. format specifiers.
> > 
> > Tested on x86_64-linux, ok for trunk?
> > 
> > 2024-10-15  Jakub Jelinek  <ja...@redhat.com>
> > 
> >     * configure.ac (gcc_AC_CHECK_DECLS): Add fmemopen.
> >     * configure: Regenerate.
> >     * config.in: Regenerate.
> >     * Makefile.in (build/genmatch.o): Add -DGENMATCH_SELFTESTS to
> >     BUILD_CPPFLAGS for stage2+ genmatch.
> >     * genmatch.cc (test_diag_vfprintf, genmatch_diag_selftests): New
> >     functions.
> >     (main): Call genmatch_diag_selftests.
> >     * pretty-print.cc (test_pp_format): Add two tests, one for %M$.*N$s
> >     and one for %M$.Ns.
> 
> I'd like to ping this patch.

OK if David doesn't have any further comments.

Richard.

> Thanks.
> 
>       Jakub
> 
> 

-- 
Richard Biener <rguent...@suse.de>
SUSE Software Solutions Germany GmbH,
Frankenstrasse 146, 90461 Nuernberg, Germany;
GF: Ivo Totev, Andrew McDonald, Werner Knoblich; (HRB 36809, AG Nuernberg)

Reply via email to