This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v2 10/12] soft-fp: Add the lack of implementation for 128 bit
- From: Joseph Myers <joseph at codesourcery dot com>
- To: Zong Li <zong at andestech dot com>
- Cc: <palmer at dabbelt dot com>, <darius at bluespec dot com>, <andrew at sifive dot com>, <dj at redhat dot com>, <rth at twiddle dot net>, <libc-alpha at sourceware dot org>, <rth7680 at gmail dot com>, <zongbox at gmail dot com>
- Date: Tue, 17 Jul 2018 22:09:42 +0000
- Subject: Re: [PATCH v2 10/12] soft-fp: Add the lack of implementation for 128 bit
- References: <cover.1531801545.git.zong@andestech.com> <fd0613d051df139ccef1f5f774ca8873c4ee4458.1531801545.git.zong@andestech.com>
On Tue, 17 Jul 2018, Zong Li wrote:
> Add the definition of macros for 8 bytes length. I only add the
> lack of implementation when building the RISC-V 32 bit port.
Please use *self-contained* commit messages (= patch submission messages)
with all relevant information from prior discussions.
My understanding from discussions of the previous patch series version is
that this issue is something to do with the combination of soft-fp fma,
ldbl-128 and 32-bit _FP_W_TYPE_SIZE, where every pair of those three is
already used by some glibc port but none uses all three together. So you
need to explain that properly in your commit message.
In my view, such patches that are not logically part of a port should be
posted separately from any patch series for the port, and before that
patch series, with the patch series then being noted as depending on them
(if it does).
I think consideration of (a revised version of) this patch is most
appropriately deferred until 2.29 (and probably likewise the new port
itself, given the associated risks and the lack of posting before the
freeze - posting before the freeze being particularly important for any
architecture-independent changes involved in a new port).
--
Joseph S. Myers
joseph@codesourcery.com