From: Mike Frysinger <vapier@gentoo.org>
Date: Tue, 19 Dec 2023 11:49:41 +0000 (-0500)
Subject: sim: frv: enable warnings in memory.c
X-Git-Tag: binutils-2_42~496
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=06f05f35856b37bc6db014d0921638932dd5a97a;p=thirdparty%2Fbinutils-gdb.git

sim: frv: enable warnings in memory.c

Fix one minor pointer-sign warning to enable warnings in general
for this file.  Reading the data as signed and then returning it
as unsigned should be functionally the same in this case.
---

diff --git a/sim/Makefile.in b/sim/Makefile.in
index 8d3bfaaa7e3..b07641ff132 100644
--- a/sim/Makefile.in
+++ b/sim/Makefile.in
@@ -2273,7 +2273,6 @@ testsuite_common_CPPFLAGS = \
 @SIM_ENABLE_ARCH_examples_TRUE@	$(SIM_COMMON_LIBS)
 
 @SIM_ENABLE_ARCH_frv_TRUE@AM_CPPFLAGS_frv = $(SIM_FRV_TRAPDUMP_FLAGS)
-@SIM_ENABLE_ARCH_frv_TRUE@AM_CFLAGS_frv_memory.o = -Wno-error
 @SIM_ENABLE_ARCH_frv_TRUE@AM_CFLAGS_frv_sem.o = -Wno-error
 @SIM_ENABLE_ARCH_frv_TRUE@nodist_frv_libsim_a_SOURCES = \
 @SIM_ENABLE_ARCH_frv_TRUE@	frv/modules.c
diff --git a/sim/frv/local.mk b/sim/frv/local.mk
index fab1c5f48cd..e4940398fd7 100644
--- a/sim/frv/local.mk
+++ b/sim/frv/local.mk
@@ -19,7 +19,6 @@
 AM_CPPFLAGS_%C% = $(SIM_FRV_TRAPDUMP_FLAGS)
 
 ## Some modules don't build cleanly yet.
-AM_CFLAGS_%C%_memory.o = -Wno-error
 AM_CFLAGS_%C%_sem.o = -Wno-error
 
 nodist_%C%_libsim_a_SOURCES = \
diff --git a/sim/frv/memory.c b/sim/frv/memory.c
index 062eeaed7ad..92644ac36bf 100644
--- a/sim/frv/memory.c
+++ b/sim/frv/memory.c
@@ -670,7 +670,7 @@ frvbf_read_imem_USI (SIM_CPU *current_cpu, PCADDR vpc)
   if (GET_HSR0_ICE (hsr0))
     {
       FRV_CACHE *cache;
-      USI value;
+      SI value;
 
       /* We don't want this to show up in the cache statistics.  That read
 	 is done in frvbf_simulate_insn_prefetch.  So read the cache or memory