libstdc++: Fix stream extraction of IEEE128 long double [PR100912]
authorJonathan Wakely <jwakely@redhat.com>
Mon, 14 Feb 2022 16:46:55 +0000 (16:46 +0000)
committerJonathan Wakely <jwakely@redhat.com>
Mon, 14 Feb 2022 17:19:08 +0000 (17:19 +0000)
commit3d50dede07de0923f0f320d385162e546445e640
tree16cc21ce2695ca23db1045a731d1ab9ee36f9850
parent02a981a8e512934a990d1427d14e8e884409fade
libstdc++: Fix stream extraction of IEEE128 long double [PR100912]

The std::__convert_from_v helper that formats double and long double
values into a char buffer was not being duplicated for the two long
double ABIs. This resulted in an ODR violation inside the library, where
some callers needed it to use snprintf to format __ibm128 values and
other callers needed it to use __snprintfieee128 to format __ieee128
values. The linker discarded one of the definitions, leaving one set of
callers using the wrong code.

This puts __convert_from_v in the __gnu_cxx_ieee128 inline namespace
when long double is __ieee128, so that there are two different
definitions of the function.

The std::money_put::__do_put overload for __ibm128 values needs a
different fix, because that is defined when long double is __ieee128 and
so would call the one in the inline namespace. That can be fixed by just
inlining the code directly into the function and using an asm alias to
call the right version of snprintf for the __ibm128 format. The code to
do that can be simpler than __convert_from_v because if we're defining
the ALT128_COMPAT symbols we know that we have a recent glibc and so we
can assume that uselocale and snprintf are supported.

libstdc++-v3/ChangeLog:

PR libstdc++/100912
* config/locale/gnu/c_locale.h (__convert_from_v): Use inline
namespace for IEEE128 long double mode.
* config/os/gnu-linux/ldbl-ieee128-extra.ver: Add new symbol
version and export __gnu_cxx_ieee128::__convert_from_v.
* include/bits/locale_facets_nonio.tcc (money_put::__do_put):
Make __ibm128 overload use snprintf directly
* testsuite/util/testsuite_abi.cc: Add new symbol version.
Remove stable IEEE128/LDBL versions.
libstdc++-v3/config/locale/gnu/c_locale.h
libstdc++-v3/config/os/gnu-linux/ldbl-ieee128-extra.ver
libstdc++-v3/include/bits/locale_facets_nonio.tcc
libstdc++-v3/testsuite/util/testsuite_abi.cc