diff options
author | Dimitar Dimitrov <dimitar@dinux.eu> | 2024-01-12 17:58:57 +0200 |
---|---|---|
committer | Dimitar Dimitrov <dimitar@dinux.eu> | 2024-01-12 21:48:25 +0200 |
commit | b83808a8a2265371ea78c05771dde3d6a6b81115 (patch) | |
tree | fcb463d9cc40b7504a2912a88ee7005a45a8c42e | |
parent | ba7c1e37cfb498263a4e309700c27696fa5ff1ad (diff) | |
download | binutils-b83808a8a2265371ea78c05771dde3d6a6b81115.zip binutils-b83808a8a2265371ea78c05771dde3d6a6b81115.tar.gz binutils-b83808a8a2265371ea78c05771dde3d6a6b81115.tar.bz2 |
sim: Fix compile errors
The following change broke simulator testsuite with host GCC 13:
commit 435ad222b3de93fa647fba7221eece36b1b395eb
sim: warnings: compile build tools with -Werror too
Host GCC13 complains about missing function prototypes:
binutils/sim/testsuite/common/bits-gen.c:26:1: error: no previous prototype for ‘gen_struct’ [-Werror=missing-prototypes]
26 | gen_struct (void)
| ^~~~~~~~~~
Fix by making the functions static, which instructs the compiler that
there is no need for a prototype.
Signed-off-by: Dimitar Dimitrov <dimitar@dinux.eu>
-rw-r--r-- | sim/testsuite/common/alu-n-tst.h | 4 | ||||
-rw-r--r-- | sim/testsuite/common/bits-gen.c | 10 | ||||
-rw-r--r-- | sim/testsuite/common/bits-tst.c | 10 |
3 files changed, 12 insertions, 12 deletions
diff --git a/sim/testsuite/common/alu-n-tst.h b/sim/testsuite/common/alu-n-tst.h index 14abe86..ffd962c 100644 --- a/sim/testsuite/common/alu-n-tst.h +++ b/sim/testsuite/common/alu-n-tst.h @@ -27,7 +27,7 @@ #define do_op_N XCONCAT2(do_op_,N) -void +static void do_op_N (const alu_test *tst) { const alu_op *op; @@ -120,7 +120,7 @@ do_op_N (const alu_test *tst) } -const alu_test alu_N_tests[] = { +static const alu_test alu_N_tests[] = { /* 0 + 0; 0 + 1; 1 + 0; 1 + 1 */ { 0, { { "ADDC", 0 }, }, 0, 0, 0, }, diff --git a/sim/testsuite/common/bits-gen.c b/sim/testsuite/common/bits-gen.c index d881875..fc1333b 100644 --- a/sim/testsuite/common/bits-gen.c +++ b/sim/testsuite/common/bits-gen.c @@ -22,7 +22,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */ #include <string.h> #include <unistd.h> -void +static void gen_struct (void) { printf ("\n"); @@ -44,7 +44,7 @@ gen_struct (void) } -void +static void gen_bit (int bitsize, int msb, const char *macro, @@ -79,7 +79,7 @@ gen_bit (int bitsize, } -void +static void gen_enum (const char *macro, int nr_bits) { @@ -96,7 +96,7 @@ gen_enum (const char *macro, } -void +static void gen_mask (int bitsize, const char *msb, const char *macro, @@ -160,7 +160,7 @@ gen_mask (int bitsize, } -void +static void usage (int reason) { fprintf (stderr, "Usage:\n"); diff --git a/sim/testsuite/common/bits-tst.c b/sim/testsuite/common/bits-tst.c index bcdee10..a30753e 100644 --- a/sim/testsuite/common/bits-tst.c +++ b/sim/testsuite/common/bits-tst.c @@ -3,7 +3,7 @@ /* Drive the bit test routines */ -long long +static long long calc (const char *call, long long val, int row, @@ -161,7 +161,7 @@ calc (const char *call, } -int +static int check_sext (int nr_bits, int msb_nr, const char *sexted, @@ -214,7 +214,7 @@ check_sext (int nr_bits, } -int +static int check_rot (int nr_bits, const char *roted, const char *masked) @@ -248,7 +248,7 @@ check_rot (int nr_bits, } -int +static int check_extract (int nr_bits, const char *extracted, const char *inserted, @@ -277,7 +277,7 @@ check_extract (int nr_bits, } -int +static int check_bits (int call, test_spec **tests) { |