[PATCH v3] kvx: New port.

[Luis Machado]

On 8/16/23 14:24, Nick Clifton via Binutils wrote:
> Hi Paul,
> 
>> Attached is the patch series revised.
> 
> Sorry for the delay in responding - I was away on PTO.
> 
> Anyway, the patch series is now in very good shape, so I have
> gone ahead and applied it.
> 
> Thanks very much for contributing to the binutils.
> 
> Cheers
>   Nick
> 
> 

The builders have caught a few hiccups related to printf modifiers for 32-bit architectures.

Could you please take a look at those? A build with --enable-targets=all is recommended
to catch those.

See https://builder.sourceware.org/buildbot/#/builders/170/builds/3977.

--

In file included from ../../binutils-gdb/bfd/elfnn-kvx.c:22:
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:74:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
   74 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:243:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
  243 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:399:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
  399 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:490:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
  490 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:737:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
  737 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:750:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
  750 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfxx-kvx-relocs.h:958:3: error: conversion from ‘long long unsigned int’ to ‘unsigned int’ changes value from ‘18446744073709551615’ to ‘4294967295’ [-Werror=overflow]
  958 |   0xffffffffffffffff,   /* dst_mask */
      |   ^~~~~~~~~~~~~~~~~~
bfd.h:3204:15: note: in definition of macro ‘HOWTO’
 3204 |     src_mask, dst_mask, func, name }
      |               ^~~~~~~~
../../binutils-gdb/bfd/elfnn-kvx.c: In function ‘elf32_kvx_stub_name’:
../../binutils-gdb/bfd/elfnn-kvx.c:740:28: error: format ‘%llx’ expects argument of type ‘long long unsigned int’, but argument 6 has type ‘bfd_vma’ {aka ‘const unsigned int’} [-Werror=format=]
  740 |  snprintf (stub_name, len, "%08x_%s+%" PRIx64 "x",
      |                            ^~~~~~~~~~~
......
  743 |     rel->r_addend);
      |     ~~~~~~~~~~~~~
      |        |
      |        bfd_vma {aka const unsigned int}
../../binutils-gdb/bfd/elfnn-kvx.c:750:28: error: format ‘%llx’ expects argument of type ‘long long unsigned int’, but argument 7 has type ‘bfd_vma’ {aka ‘const unsigned int’} [-Werror=format=]
  750 |  snprintf (stub_name, len, "%08x_%x:%x+%" PRIx64 "x",
      |                            ^~~~~~~~~~~~~~
......
  754 |     rel->r_addend);
      |     ~~~~~~~~~~~~~
      |        |
      |        bfd_vma {aka const unsigned int}
cc1: all warnings being treated as errors
make[3]: *** [Makefile:1758: elf32-kvx.lo] Error 1

--