This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Ping. 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, 10 Apr 2019 17:52:09 -0300
- Subject: Ping. Re: [PATCH v4] Prepare vfprintf to use __printf_fp/__printf_fphex with float128 arg
- References: <20190225133946.17585-1-gabriel@inconstante.eti.br>
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.