Ping^3. Re: [PATCH v4] Prepare vfprintf to use __printf_fp/__printf_fphex with float128 arg

Gabriel F. T. Gomes gabriel@inconstante.eti.br
Fri May 3 02:05:00 GMT 2019


Ping.  This patch is pending review.
https://sourceware.org/ml/libc-alpha/2019-02/msg00582.html

On Wed, Apr 24 2019, Gabriel F. T. Gomes wrote:
> 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.



More information about the Libc-alpha mailing list