[binutils-gdb] sim: nrun: tweak init of callback endian

Michael Frysinger vapier@sourceware.org
Wed Jun 9 23:05:46 GMT 2021


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=cfc6061bd857d4b2e90d00a3bc2bbac038874f51

commit cfc6061bd857d4b2e90d00a3bc2bbac038874f51
Author: Mike Frysinger <vapier@gentoo.org>
Date:   Sat Jun 5 22:50:57 2021 -0400

    sim: nrun: tweak init of callback endian
    
    Allow ports to initialize the callback endian if they want.  This will
    allow delegation of the logic out of common code in the future.
    
    Also switch from the CURRENT_TARGET_BYTE_ORDER macro to the underlying
    current_target_byte_order storage since the latter has been setup by
    the sim-config module based on the same macros.  This will allow the
    nrun module to be moved to common building for sharing.

Diff:
---
 sim/common/ChangeLog |  5 +++++
 sim/common/nrun.c    | 11 ++++++-----
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index dbe9236056d..b5b6db2f9a2 100644
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,8 @@
+2021-06-09  Mike Frysinger  <vapier@gentoo.org>
+
+	* nrun.c (main): Set default_callback.target_endian to
+	current_target_byte_order when it's BFD_ENDIAN_UNKNOWN.
+
 2021-06-09  Mike Frysinger  <vapier@gentoo.org>
 
 	* cgen-defs.h (STATE_RUN_FAST_P, CGEN_STATE, cgen_init): Delete.
diff --git a/sim/common/nrun.c b/sim/common/nrun.c
index 2cccff6a211..e39a0c6ed54 100644
--- a/sim/common/nrun.c
+++ b/sim/common/nrun.c
@@ -88,11 +88,12 @@ main (int argc, char **argv)
       abort ();
     }
 
-  /* We can't set the endianness in the callback structure until
-     sim_config is called, which happens in sim_open.  */
-  default_callback.target_endian
-    = (CURRENT_TARGET_BYTE_ORDER == BFD_ENDIAN_BIG
-       ? BFD_ENDIAN_BIG : BFD_ENDIAN_LITTLE);
+  /* We can't set the endianness in the callback structure until sim_config is
+     called, which happens in sim_open.  If it's still the default, switch it.
+     Don't use CURRENT_TARGET_BYTE_ORDER as all its internal processing already
+     happened in sim_config.  */
+  if (default_callback.target_endian == BFD_ENDIAN_UNKNOWN)
+    default_callback.target_endian = current_target_byte_order;
 
   /* Was there a program to run?  */
   prog_argv = STATE_PROG_ARGV (sd);


More information about the Gdb-cvs mailing list