Re: int128_t and uint128_t typeinfo

I had to commit a follow-up fix in r314104: when C++ names are used in
the version script, they have to be surrounded by an extern "C++" {}
block, otherwise the symbols end up as locals in the final library, and
thus get stripped out of the installed version.

-Dimitry

On 22 Feb 2017, at 16:19, Hartmut.Brandt_at_dlr.de wrote:
> 
> Looks like they are still not there. I've rebuilt world.
> 
> nm -D -C /usr/lib/libcxxrt.so  | grep 128
> 
> should show me the symbols, right? It does not.
> 
> harti
> 
> -----Original Message-----
> From: Dimitry Andric [mailto:dim_at_FreeBSD.org]
> Sent: Tuesday, February 21, 2017 10:52 PM
> To: Brandt, Hartmut
> Cc: current_at_freebsd.org
> Subject: Re: int128_t and uint128_t typeinfo
> 
> On 21 Feb 2017, at 18:26, Dimitry Andric <dim_at_FreeBSD.org> wrote:
>> 
>> On 21 Feb 2017, at 13:48, Hartmut Brandt <hartmut.brandt_at_dlr.de> wrote:
>>> 
>>> it looks like the typeinfo for __int128_t and __uint128_t is missing from our dynamically linked libcxxrt.
> ...
>> * We also need to add the typeinfo for __u?int128_t * and __u?int128_t
>> const *
>> * Maybe these should be under the CXXABI_2.0 version, since that is
>> where newer libstdc++ places them
>> * Maybe these should be dependent on whether the architecture supports
>> 128 bit integers at all
>> 
>> I need to think a bit on the above, then I'll commit a fix.
> 
> Okay, can you please try r314061?
> 
> -Dimitry
>