From 47c3ed988be2da61c38d896925b8cede2fd67808 Mon Sep 17 00:00:00 2001 From: "Kaveh R. Ghazi" Date: Mon, 18 Jan 1999 06:17:31 +0000 Subject: [PATCH] Warning fixes: * loop.c (insert_bct): Hide the definition of variables `increment_direction', `compare_direction', `add_iteration' and `loop_var_mode'. * recog.c (mode_dependent_address_p): Mark parameter `addr' with ATTRIBUTE_UNUSED. Mark label `win' with ATTRIBUTE_UNUSED_LABEL. (mode_independent_operand): Mark label `lose' with ATTRIBUTE_UNUSED_LABEL. * regclass.c (n_occurrences): Remove prototype and definition. * reload.c (find_reloads_address_1): Mark variable `tem' with ATTRIBUTE_UNUSED. * reload1.c (reload): Cast the first two arguments of `bcopy' to PTR. * sbitmap.c (sbitmap_copy): Likewise. * scan-decls.c (scan_decls): Hide label `handle_comma'. * toplev.c (output_lang_identify): Mark prototype with ATTRIBUTE_UNUSED. * tree.c (make_node): Cast the first argument of `bzero' to PTR. (make_tree_vec): Likewise. (build1): Likewise. * varasm.c (assemble_static_space): Mark variable `tem' with ATTRIBUTE_UNUSED. From-SVN: r24740 --- gcc/ChangeLog | 32 ++++++++++++++++++++++++++++++++ gcc/loop.c | 2 ++ gcc/recog.c | 8 +++++--- gcc/regclass.c | 13 ------------- gcc/reload.c | 5 +++-- gcc/reload1.c | 2 +- gcc/sbitmap.c | 3 ++- gcc/scan-decls.c | 4 +++- gcc/toplev.c | 3 ++- gcc/tree.c | 6 +++--- gcc/varasm.c | 10 ++++++---- 11 files changed, 59 insertions(+), 29 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index fd680bfb184..8f1b424d8cb 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,35 @@ +Mon Jan 18 09:05:37 1999 Kaveh R. Ghazi + + * loop.c (insert_bct): Hide the definition of variables + `increment_direction', `compare_direction', `add_iteration' and + `loop_var_mode'. + + * recog.c (mode_dependent_address_p): Mark parameter `addr' with + ATTRIBUTE_UNUSED. Mark label `win' with ATTRIBUTE_UNUSED_LABEL. + (mode_independent_operand): Mark label `lose' with + ATTRIBUTE_UNUSED_LABEL. + + * regclass.c (n_occurrences): Remove prototype and definition. + + * reload.c (find_reloads_address_1): Mark variable `tem' with + ATTRIBUTE_UNUSED. + + * reload1.c (reload): Cast the first two arguments of `bcopy' to PTR. + + * sbitmap.c (sbitmap_copy): Likewise. + + * scan-decls.c (scan_decls): Hide label `handle_comma'. + + * toplev.c (output_lang_identify): Mark prototype with + ATTRIBUTE_UNUSED. + + * tree.c (make_node): Cast the first argument of `bzero' to PTR. + (make_tree_vec): Likewise. + (build1): Likewise. + + * varasm.c (assemble_static_space): Mark variable `tem' with + ATTRIBUTE_UNUSED. + Mon Jan 18 04:28:36 1999 Nathan Sidwell * Makefile.in (GCOV_INSTALL_NAME): New macro. diff --git a/gcc/loop.c b/gcc/loop.c index c3e20b7cccc..c379b6162d4 100644 --- a/gcc/loop.c +++ b/gcc/loop.c @@ -7944,6 +7944,7 @@ insert_bct (loop_start, loop_end, loop_info) int i; unsigned HOST_WIDE_INT n_iterations; +#if 0 int increment_direction, compare_direction; /* If the loop condition is <= or >=, the number of iteration @@ -7951,6 +7952,7 @@ insert_bct (loop_start, loop_end, loop_info) int add_iteration = 0; enum machine_mode loop_var_mode = word_mode; +#endif int loop_num = uid_loop_num [INSN_UID (loop_start)]; diff --git a/gcc/recog.c b/gcc/recog.c index 0d87386cdb7..8804ae91ab2 100644 --- a/gcc/recog.c +++ b/gcc/recog.c @@ -1861,11 +1861,12 @@ offsettable_address_p (strictp, mode, y) int mode_dependent_address_p (addr) - rtx addr; + rtx addr ATTRIBUTE_UNUSED; /* Maybe used in GO_IF_MODE_DEPENDENT_ADDRESS. */ { GO_IF_MODE_DEPENDENT_ADDRESS (addr, win); return 0; - win: + /* Label `win' might (not) be used via GO_IF_MODE_DEPENDENT_ADDRESS. */ + win: ATTRIBUTE_UNUSED_LABEL return 1; } @@ -1888,7 +1889,8 @@ mode_independent_operand (op, mode) addr = XEXP (op, 0); GO_IF_MODE_DEPENDENT_ADDRESS (addr, lose); return 1; - lose: + /* Label `lose' might (not) be used via GO_IF_MODE_DEPENDENT_ADDRESS. */ + lose: ATTRIBUTE_UNUSED_LABEL return 0; } diff --git a/gcc/regclass.c b/gcc/regclass.c index 2ba93d82f1f..5b65f235eea 100644 --- a/gcc/regclass.c +++ b/gcc/regclass.c @@ -691,7 +691,6 @@ static int loop_depth; static int loop_cost; -static int n_occurrences PROTO((int, char *)); static rtx scan_one_insn PROTO((rtx, int)); static void record_reg_classes PROTO((int, int, rtx *, enum machine_mode *, char **, rtx)); @@ -742,18 +741,6 @@ regclass_init () prefclass = 0; } -/* Return the number of times character C occurs in string S. */ -static int -n_occurrences (c, s) - int c; - char *s; -{ - int n = 0; - while (*s) - n += (*s++ == c); - return n; -} - /* Subroutine of regclass, processes one insn INSN. Scan it and record each time it would save code to put a certain register in a certain class. PASS, when nonzero, inhibits some optimizations which need only be done diff --git a/gcc/reload.c b/gcc/reload.c index 4ab2a283dc0..4bbf317d684 100644 --- a/gcc/reload.c +++ b/gcc/reload.c @@ -1,5 +1,5 @@ /* Search an insn for pseudo regs that must be in hard regs and are not. - Copyright (C) 1987, 88, 89, 92-97, 1998 Free Software Foundation, Inc. + Copyright (C) 1987, 88, 89, 92-98, 1999 Free Software Foundation, Inc. This file is part of GNU CC. @@ -5318,7 +5318,8 @@ find_reloads_address_1 (mode, x, context, loc, opnum, type, ind_levels, insn) of an equivalent address for a pseudo that was not allocated to a hard register. Verify that the specified address is valid and reload it into a register. */ - rtx tem = XEXP (x, 0); + /* Variable `tem' might or might not be used in FIND_REG_INC_NOTE. */ + rtx tem ATTRIBUTE_UNUSED = XEXP (x, 0); register rtx link; int reloadnum; diff --git a/gcc/reload1.c b/gcc/reload1.c index 1c53f013bb4..6a6ceb63dfb 100644 --- a/gcc/reload1.c +++ b/gcc/reload1.c @@ -662,7 +662,7 @@ reload (first, global, dumpfile) reg_max_ref_width = (int *) xmalloc (max_regno * sizeof (int)); bzero ((char *) reg_max_ref_width, max_regno * sizeof (int)); reg_old_renumber = (short *) xmalloc (max_regno * sizeof (short)); - bcopy (reg_renumber, reg_old_renumber, max_regno * sizeof (short)); + bcopy ((PTR) reg_renumber, (PTR) reg_old_renumber, max_regno * sizeof (short)); pseudo_forbidden_regs = (HARD_REG_SET *) xmalloc (max_regno * sizeof (HARD_REG_SET)); pseudo_previous_regs diff --git a/gcc/sbitmap.c b/gcc/sbitmap.c index db47d327ea5..6513d077ecd 100644 --- a/gcc/sbitmap.c +++ b/gcc/sbitmap.c @@ -96,7 +96,8 @@ void sbitmap_copy (dst, src) sbitmap dst, src; { - bcopy (src->elms, dst->elms, sizeof (SBITMAP_ELT_TYPE) * dst->size); + bcopy ((PTR) src->elms, (PTR) dst->elms, + sizeof (SBITMAP_ELT_TYPE) * dst->size); } /* Zero all elements in a bitmap. */ diff --git a/gcc/scan-decls.c b/gcc/scan-decls.c index 2c6f12ae7b4..5fea64309b4 100644 --- a/gcc/scan-decls.c +++ b/gcc/scan-decls.c @@ -1,5 +1,5 @@ /* scan-decls.c - Extracts declarations from cpp output. - Copyright (C) 1993, 1995, 1997, 1998 Free Software Foundation, Inc. + Copyright (C) 1993, 1995, 97-98, 1999 Free Software Foundation, Inc. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by the @@ -189,7 +189,9 @@ scan_decls (pfile, argc, argv) maybe_handle_comma: if (token != CPP_COMMA) goto new_statement; +#if 0 handle_comma: +#endif /* Handle multiple declarators in a single declaration, as in: extern char *strcpy (), *strcat (), ... ; */ if (declarator_start == 0) diff --git a/gcc/toplev.c b/gcc/toplev.c index 5bd82dcb9dd..a58e16effda 100644 --- a/gcc/toplev.c +++ b/gcc/toplev.c @@ -197,7 +197,8 @@ static void v_really_sorry PROTO((char *, va_list)) ATTRIBUTE_NORETURN; static void float_signal PROTO((int)) ATTRIBUTE_NORETURN; static void pipe_closed PROTO((int)) ATTRIBUTE_NORETURN; #ifdef ASM_IDENTIFY_LANGUAGE -static void output_lang_identify PROTO((FILE *)); +/* This might or might not be used in ASM_IDENTIFY_LANGUAGE. */ +static void output_lang_identify PROTO((FILE *)) ATTRIBUTE_UNUSED; #endif static void open_dump_file PROTO((char *, char *)); static void close_dump_file PROTO((void (*) (FILE *, rtx), rtx)); diff --git a/gcc/tree.c b/gcc/tree.c index 053420cd481..f7db3c4d90f 100644 --- a/gcc/tree.c +++ b/gcc/tree.c @@ -1070,7 +1070,7 @@ make_node (code) } t = (tree) obstack_alloc (obstack, length); - bzero (t, length); + bzero ((PTR) t, length); #ifdef GATHER_STATISTICS tree_node_counts[(int)kind]++; @@ -1551,7 +1551,7 @@ make_tree_vec (len) #endif t = (tree) obstack_alloc (obstack, length); - bzero (t, length); + bzero ((PTR) t, length); TREE_SET_CODE (t, TREE_VEC); TREE_VEC_LENGTH (t) = len; @@ -3058,7 +3058,7 @@ build1 (code, type, node) length = sizeof (struct tree_exp); t = (tree) obstack_alloc (obstack, length); - bzero (t, length); + bzero ((PTR) t, length); #ifdef GATHER_STATISTICS tree_node_counts[(int)kind]++; diff --git a/gcc/varasm.c b/gcc/varasm.c index 343c60364c6..265987a4f3c 100644 --- a/gcc/varasm.c +++ b/gcc/varasm.c @@ -1,5 +1,5 @@ /* Output variables, constants and external declarations, for GNU compiler. - Copyright (C) 1987, 88, 89, 92-97, 1998 Free Software Foundation, Inc. + Copyright (C) 1987, 88, 89, 92-98, 1999 Free Software Foundation, Inc. This file is part of GNU CC. @@ -1711,9 +1711,11 @@ assemble_static_space (size) { /* Round size up to multiple of BIGGEST_ALIGNMENT bits so that each uninitialized object starts on such a boundary. */ - int rounded = ((size + (BIGGEST_ALIGNMENT / BITS_PER_UNIT) - 1) - / (BIGGEST_ALIGNMENT / BITS_PER_UNIT) - * (BIGGEST_ALIGNMENT / BITS_PER_UNIT)); + /* Variable `rounded' might or might not be used in ASM_OUTPUT_LOCAL. */ + int rounded ATTRIBUTE_UNUSED + = ((size + (BIGGEST_ALIGNMENT / BITS_PER_UNIT) - 1) + / (BIGGEST_ALIGNMENT / BITS_PER_UNIT) + * (BIGGEST_ALIGNMENT / BITS_PER_UNIT)); ASM_OUTPUT_LOCAL (asm_out_file, name, size, rounded); } #endif