This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Ping^2. Re: [PATCH v4] Prepare vfprintf to use __printf_fp/__printf_fphex with float128 arg
- From: "Gabriel F. T. Gomes" <gabriel at inconstante dot eti dot br>
- To: <libc-alpha at sourceware dot org>
- Date: Wed, 24 Apr 2019 22:59:50 -0300
- Subject: Ping^2. Re: [PATCH v4] Prepare vfprintf to use __printf_fp/__printf_fphex with float128 arg
- References: <20190225133946.17585-1-gabriel@inconstante.eti.br> <20190410205209.h7lpqpzssmwfyyxf@tereshkova.br.ibm.com>
Ping. This patch is pending review.
On Wed, Apr 10 2019, Gabriel F. T. Gomes wrote:
> Ping. This patch [1] is pending review.
>
> Testing this patch depends on the subsequent patches that create new
> functions, e.g. __printfieee128, that actually use these changes, as
> well as it depends on the subsequent patch that enables the ldbl
> redirections (e.g. from printf to __printfieee128 in bits/stdio-ldbl.h).
>
> However, such redirections can only be turned on when the functions for
> all other redirections (i.e. those in *-ldbl.h headers) are ready.
> Thus, I uploaded all of them to my personal branch [2], in the hopes
> that someone willing to understand how this patch is used, *and tested*,
> can do it easier (I haven't simply submitted all the patches from my
> branch to the mailing list, because I think some of them still need a
> little work from me before review).
>
> [1] https://sourceware.org/ml/libc-alpha/2019-02/msg00582.html
> [2] https://sourceware.org/git/?p=glibc.git;a=shortlog;h=refs/heads/gabriel/powerpc-ieee128-printscan
>
> On Mon, Feb 25 2019, Gabriel F. T. Gomes wrote:
> > Changes since v3:
> >
> > - Update after the commit 2d9837c1fbf4 ('Set behavior of sprintf-like
> > functions with overlapping source and destination').
> >
> > Changes since v2:
> >
> > - Fixed style error in `do { ... } while (0)' blocks.
> > - Zero-initialize args_value[cnt] with memset, rather than relying on
> > the `.pa_long_double' member being the largest of the members.
> >
> > Changes since v1:
> >
> > - Updated to the revised and integrated patches for __ldbl_is_dbl
> > removal, i.e.: the patches in the following thread:
> > <https://sourceware.org/ml/libc-alpha/2018-12/msg00186.html>.
> > - Added description for the PRINTF_LDBL_USES_FLOAT128 macro.
> > - Removed the LDBL_USES_FLOAT128 macro.
> > - Added `do { } while (0)' to the PARSE_FLOAT_VA_ARG_EXTENDED,
> > PARSE_FLOAT_VA_ARG, and SETUP_FLOAT128_INFO macros. Appended
> > expansions with `;', accordingly.