diff mbox series

[37/40] sim/sh: Remove redundant function declaration

Message ID 0db6b0934795da17889f71fb469ab81183fbf3c6.1666258361.git.research_trasio@irq.a4lg.com
State Committed
Commit 9b77569146a8ec354a6010497049428d2a9ceb1d
Headers
Series sim+gdb: Suppress warnings if built with Clang (big batch 1) |

Commit Message

Tsukasa OI Oct. 20, 2022, 9:32 a.m. UTC 
  Clang generates a warning if there is a function declaration/definition
with zero arguments.  Such declarations/definitions without a prototype (an
argument list) are deprecated forms of indefinite arguments
("-Wdeprecated-non-prototype").  On the default configuration, it causes a
build failure (unless "--disable-werror" is specified).

But there is another issue.  This function declaration in sim/sh/interp.c
is completely redundant.  This commit just removes that declaration.
---
 sim/sh/interp.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Mike Frysinger Oct. 23, 2022, 2:19 p.m. UTC | #1 
lgtm
-mike
diff mbox series

Patch

diff --git a/sim/sh/interp.c b/sim/sh/interp.c
index 38f3f945a35..b6f29880d74 100644
--- a/sim/sh/interp.c
+++ b/sim/sh/interp.c
@@ -1492,8 +1492,6 @@  get_loop_bounds (int rs, int re, unsigned char *memory, unsigned char *mem_end,
   return loop;
 }
 
-static void ppi_insn ();
-
 #include "ppi.c"
 
 /* Provide calloc / free versions that use an anonymous mmap.  This can