aboutsummaryrefslogtreecommitdiff
path: root/sim
diff options
context:
space:
mode:
authorTom Tromey <tom@tromey.com>2021-04-08 15:15:59 -0600
committerTom Tromey <tom@tromey.com>2021-04-08 15:15:59 -0600
commit06a88b3b3948db72b662345ac7abf90bd1c14a44 (patch)
treefc3f3e4e0f00a2c6c7b4ac88764399130bf625e2 /sim
parent32a046ab0d7717256f16f59d87438474e825fcb8 (diff)
downloadgdb-06a88b3b3948db72b662345ac7abf90bd1c14a44.zip
gdb-06a88b3b3948db72b662345ac7abf90bd1c14a44.tar.gz
gdb-06a88b3b3948db72b662345ac7abf90bd1c14a44.tar.bz2
Avoid sequence point warning in h8300 sim
GCC gives a -Wsequence-point warning for this code in the h8300 sim. The bug is that memory_size is both assigned and used in the same expression. The fix is to assign after the print. sim/h8300/ChangeLog 2021-04-08 Tom Tromey <tom@tromey.com> * compile.c (init_pointers): Fix sequence point warning.
Diffstat (limited to 'sim')
-rw-r--r--sim/h8300/ChangeLog4
-rw-r--r--sim/h8300/compile.c3
2 files changed, 6 insertions, 1 deletions
diff --git a/sim/h8300/ChangeLog b/sim/h8300/ChangeLog
index cd47d25..2b2c6a7 100644
--- a/sim/h8300/ChangeLog
+++ b/sim/h8300/ChangeLog
@@ -1,5 +1,9 @@
2021-04-08 Tom Tromey <tom@tromey.com>
+ * compile.c (init_pointers): Fix sequence point warning.
+
+2021-04-08 Tom Tromey <tom@tromey.com>
+
* compile.c (cmdline_location): Use new-style declaration.
2021-04-02 Mike Frysinger <vapier@gentoo.org>
diff --git a/sim/h8300/compile.c b/sim/h8300/compile.c
index c30a738..c8060b2 100644
--- a/sim/h8300/compile.c
+++ b/sim/h8300/compile.c
@@ -1624,7 +1624,8 @@ init_pointers (SIM_DESC sd)
sim_io_printf
(sd,
"init_pointers: bad memory size %d, defaulting to %d.\n",
- memory_size, memory_size = H8300S_MSIZE);
+ memory_size, H8300S_MSIZE);
+ memory_size = H8300S_MSIZE;
}
if (h8_get_memory_buf (sd))