[PATCH][binutils][Arm][2/3] Add support for float16 (IEEE and Alternative formats) for Arm backend.

Nick Clifton nickc@redhat.com
Fri Aug 9 12:31:00 GMT 2019


Hi Barnaby,

> Cross compiled and regtested on arm-none-eabi and arm-none-linux-gnueabihf

Are you sure ?  I am seeing the following new assembler failures when I test the patch:

  FAIL: Invalid float16 literals (IEEE 754 & Alternative)
  FAIL: Big endian float16 literals (IEEE 754 & Alternative)
  FAIL: Tag_ABI_FP_16bit_format EABI attribute written for Arm alternative format.
  FAIL: Tag_ABI_FP_16bit_format written for IEEE float16 format.
  FAIL: Tag_ABI_FP_16bit_format EABI attribute not written when format not specified
  FAIL: Invalid combination of command line arguments and directives
  FAIL: Little endian float16 literals (IEEE 754 & Alternative)

(This is with lots of different configurations of ARM based toolchains).

The problem seems to be the same with all of the tests.  Eg:

  gas/testsuite/gas/arm/float16.s: Assembler messages:
  gas/testsuite/gas/arm/float16.s:2: Error: cannot create floating-point number
  gas/testsuite/gas/arm/float16.s:2: Error: junk at end of line, first unrecognized character is `1'

It looks like the patch is maybe missing a change to atof_ieee.c to handle the 'h'
format maybe ?

Cheers
  Nick



More information about the Binutils mailing list