[committed,2/6] Gold/MIPS: Use EM_MIPS instead of EM_MIPS_RS3_LE for little endian

Message ID 20230828034940.2056275-2-yunqiang.su@cipunited.com
State Accepted
Headers
Series [committed,1/6] Gold: Add targ_extra_little_endian to configure.ac |

Checks

Context Check Description
snail/binutils-gdb-check success Github commit url

Commit Message

YunQiang Su Aug. 28, 2023, 3:49 a.m. UTC
  EM_MIPS_RS3_LE has been deprecated quite long ago, and in fact
most of current LE ELF files are using EM_MIPS.

This problem didn't make some trouble for us, is due to that
gold is a linker, and all of the inputs to it has right EM values.
---
 gold/configure.tgt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

YunQiang Su Oct. 7, 2023, 3:32 a.m. UTC | #1
@Maciej W. Rozycki Since this patch is so simple, so is it ok to recommit it?
  
Maciej W. Rozycki Oct. 7, 2023, 3:01 p.m. UTC | #2
On Sat, 7 Oct 2023, YunQiang Su wrote:

> @Maciej W. Rozycki Since this patch is so simple, so is it ok to recommit it?

 As I requested off-list please resubmit any changes from this series you 
want to have reviewed.

  Maciej
  

Patch

diff --git a/gold/configure.tgt b/gold/configure.tgt
index b472b8f0bad..996bc901d1d 100644
--- a/gold/configure.tgt
+++ b/gold/configure.tgt
@@ -155,7 +155,7 @@  aarch64*-*)
  ;;
 mips*el*-*-*|mips*le*-*-*)
  targ_obj=mips
- targ_machine=EM_MIPS_RS3_LE
+ targ_machine=EM_MIPS
  targ_size=32
  targ_big_endian=false
  targ_extra_big_endian=true