Re: [PATCH 2/2] gdb: Enable finish command and inferior calls for _Float16 on amd64 and i386.

[Simon Marchi via Gdb-patches <gdb-patches@sourceware.org>]

On 2021-07-30 8:04 a.m., Luis Machado via Gdb-patches wrote:
> On 7/30/21 4:52 AM, Willgerodt, Felix wrote:
>>> Is _Float16 equivalent to fp16? If so, the presence of yet another name for a
>>> fp16 type is a bit unfortunate.
>>>
>>
>> They are the same usually, but on ARM the story is apparently a bit more complex:
>> https://gcc.gnu.org/onlinedocs/gcc/Half-Precision.html
> 
> It is. :-)
> 
>>
>> But to cite the link:
>> " The __fp16 type may only be used as an argument to intrinsics defined in
>> <arm_fp16.h>, or as a storage format."
>>
>> And:
>> "It is recommended that portable code use the _Float16 type defined by
>> ISO/IEC TS 18661-3:2015."
>>
>> Therefore I don't think it is that unfortunate and not really negotiable by now.
>> Both GCC and clang know about both already.
> 
> The unfortunate bit is the lack of enough DWARF information to make the distinction between different float formats of the same size. If the compilers are OK with the naming and already support it, it's not a big deal then.

I didn't find anything else to change in addition to what Luis already
reported, so the patch is OK what that fixed.

Simon