On 1/31/25 10:13 AM, Aleksandar Rakic wrote:
From: Robert Suchanek <robert.sucha...@imgtec.com>

Remove single-float and short-double axes from multilib spec.

The single-float/short-double combination is not immediately supportable
from GCC 6 as the -fshort-double option has been removed and we do not
have backend logic to implement a direct replacement. If/when we do this
then it needs appropriate ABI markers to describe the additional
variant.

Remove final remnant of single/short config.

Add the mips32r2 mips16 little endian soft-float multilib.

Add big-endian, MIPS64R6, soft-float, N32/N64 Linux libs.

Add MIPS32R1 HF LE Linux libraries.

Add big endian microMIPSr2 hard/soft float support.

Disable microMIPSr6 multilib configs.

Cherry-picked 2b2481cc71284ad9db3dff60bd6cab2be678e87e,
0e3416279af1417b85d1a09b1e74327c31899a5d,
e50ab07265fd8188bd4275c14b744ed2dc39116d,
32f7098d7d5bee9754c7728639a0e1cdb24d63f7,
24e261b2c9a9bea1c205cfab761c218ad50f938e, and
796ddebed418e953ba7cd5de1da42311fb1fe096
from https://github.com/MIPS/gcc

Signed-off-by: Robert Suchanek <robert.sucha...@imgtec.com>
Signed-off-by: Matthew Fortune <matthew.fort...@imgtec.com>
Signed-off-by: Chao-ying Fu <c...@mips.com>
Signed-off-by: Faraz Shahbazker <fshahbaz...@wavecomp.com>
Signed-off-by: Aleksandar Rakic <aleksandar.ra...@htecgroup.com>
---
  config-ml.in                 |  25 ++-
  configure                    |  25 +++
  configure.ac                 |  25 +++
  gcc/Makefile.in              |  20 ++
  gcc/config.gcc               |  12 +-
  gcc/config/mips/ml-img-elf   |  12 +
  gcc/config/mips/ml-img-linux |  10 +
  gcc/config/mips/ml-mti-elf   |  31 +++
  gcc/config/mips/ml-mti-linux |  27 +++
  gcc/config/mips/mti-elf.h    |   2 +
  gcc/config/mips/mti-linux.h  |   2 +
  gcc/config/mips/t-img-elf    |  33 ---
  gcc/config/mips/t-img-linux  |  38 ----
  gcc/config/mips/t-mips-multi | 409 +++++++++++++++++++++++++++++++++++
  gcc/config/mips/t-mti-elf    |  48 ----
  gcc/config/mips/t-mti-linux  | 158 --------------
  gcc/configure                |   8 +-
  gcc/configure.ac             |   3 +
  gcc/genmultilib              |   3 -
  19 files changed, 604 insertions(+), 287 deletions(-)
  create mode 100644 gcc/config/mips/ml-img-elf
  create mode 100644 gcc/config/mips/ml-img-linux
  create mode 100644 gcc/config/mips/ml-mti-elf
  create mode 100644 gcc/config/mips/ml-mti-linux
  delete mode 100644 gcc/config/mips/t-img-elf
  delete mode 100644 gcc/config/mips/t-img-linux
  create mode 100644 gcc/config/mips/t-mips-multi
  delete mode 100644 gcc/config/mips/t-mti-elf
  delete mode 100644 gcc/config/mips/t-mti-linux
So I'm not at all concerned about the mips specific bits of this patch. After all, they only affect mips ports and the changes seem sensible. They would need a ChangeLog entry to go forward through.

What is concerning is the config.ml change which has no comments about what it's doing or justification in the cover ltter.

Similarly it's not clear why we need a blob of mips specific code in configure.ac and the files autogenerated from that.

Jeff

Reply via email to