From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1851) id 32B333858404; Mon, 18 Oct 2021 08:17:04 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 32B333858404 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Martin Liska To: gcc-cvs@gcc.gnu.org Subject: [gcc r12-4465] Remove unused but set variables. X-Act-Checkin: gcc X-Git-Author: Martin Liska X-Git-Refname: refs/heads/master X-Git-Oldrev: 017665f63047ce47b087b0b283548a60e5abf3d2 X-Git-Newrev: 724e27046bc3f1b24eb4e153f71da0ac1049127d Message-Id: <20211018081704.32B333858404@sourceware.org> Date: Mon, 18 Oct 2021 08:17:04 +0000 (GMT) X-BeenThere: gcc-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Oct 2021 08:17:04 -0000 https://gcc.gnu.org/g:724e27046bc3f1b24eb4e153f71da0ac1049127d commit r12-4465-g724e27046bc3f1b24eb4e153f71da0ac1049127d Author: Martin Liska Date: Mon Oct 18 09:27:10 2021 +0200 Remove unused but set variables. Reported by clang13 -Wunused-but-set-variable: gcc/ChangeLog: * dbgcnt.c (dbg_cnt_process_opt): Remove unused but set variable. * gcov.c (get_cycles_count): Likewise. * lto-compress.c (lto_compression_zlib): Likewise. (lto_uncompression_zlib): Likewise. * targhooks.c (default_pch_valid_p): Likewise. libcpp/ChangeLog: * charset.c (convert_oct): Remove unused but set variable. Diff: --- gcc/dbgcnt.c | 2 -- gcc/gcov.c | 4 +--- gcc/lto-compress.c | 4 ---- gcc/targhooks.c | 4 +--- libcpp/charset.c | 2 -- 5 files changed, 2 insertions(+), 14 deletions(-) diff --git a/gcc/dbgcnt.c b/gcc/dbgcnt.c index 6a7eb34cd3e..458341a53a0 100644 --- a/gcc/dbgcnt.c +++ b/gcc/dbgcnt.c @@ -208,7 +208,6 @@ void dbg_cnt_process_opt (const char *arg) { char *str = xstrdup (arg); - unsigned int start = 0; auto_vec tokens; for (char *next = strtok (str, ","); next != NULL; next = strtok (NULL, ",")) @@ -227,7 +226,6 @@ dbg_cnt_process_opt (const char *arg) if (!dbg_cnt_process_single_pair (name, ranges[j])) break; } - start += strlen (tokens[i]) + 1; } } diff --git a/gcc/gcov.c b/gcc/gcov.c index 829e955a63b..3672ae7a6f8 100644 --- a/gcc/gcov.c +++ b/gcc/gcov.c @@ -843,7 +843,6 @@ get_cycles_count (line_info &linfo) Therefore, operating on a permuted order (i.e., non-sorted) only has the effect of permuting the output cycles. */ - bool loop_found = false; gcov_type count = 0; for (vector::iterator it = linfo.blocks.begin (); it != linfo.blocks.end (); it++) @@ -851,8 +850,7 @@ get_cycles_count (line_info &linfo) arc_vector_t path; block_vector_t blocked; vector block_lists; - loop_found |= circuit (*it, path, *it, blocked, block_lists, linfo, - count); + circuit (*it, path, *it, blocked, block_lists, linfo, count); } return count; diff --git a/gcc/lto-compress.c b/gcc/lto-compress.c index b5f4916b139..c40a13c8446 100644 --- a/gcc/lto-compress.c +++ b/gcc/lto-compress.c @@ -250,7 +250,6 @@ lto_compression_zlib (struct lto_compression_stream *stream) const size_t outbuf_length = Z_BUFFER_LENGTH; unsigned char *outbuf = (unsigned char *) xmalloc (outbuf_length); z_stream out_stream; - size_t compressed_bytes = 0; int status; gcc_assert (stream->is_compression); @@ -282,7 +281,6 @@ lto_compression_zlib (struct lto_compression_stream *stream) stream->callback ((const char *) outbuf, out_bytes, stream->opaque); lto_stats.num_compressed_il_bytes += out_bytes; - compressed_bytes += out_bytes; cursor += in_bytes; remaining -= in_bytes; @@ -342,7 +340,6 @@ lto_uncompression_zlib (struct lto_compression_stream *stream) size_t remaining = stream->bytes; const size_t outbuf_length = Z_BUFFER_LENGTH; unsigned char *outbuf = (unsigned char *) xmalloc (outbuf_length); - size_t uncompressed_bytes = 0; gcc_assert (!stream->is_compression); timevar_push (TV_IPA_LTO_DECOMPRESS); @@ -378,7 +375,6 @@ lto_uncompression_zlib (struct lto_compression_stream *stream) stream->callback ((const char *) outbuf, out_bytes, stream->opaque); lto_stats.num_uncompressed_il_bytes += out_bytes; - uncompressed_bytes += out_bytes; cursor += in_bytes; remaining -= in_bytes; diff --git a/gcc/targhooks.c b/gcc/targhooks.c index cbbcedf790f..812bbe3f16e 100644 --- a/gcc/targhooks.c +++ b/gcc/targhooks.c @@ -2200,7 +2200,7 @@ pch_option_mismatch (const char *option) /* Default version of pch_valid_p. */ const char * -default_pch_valid_p (const void *data_p, size_t len) +default_pch_valid_p (const void *data_p, size_t len ATTRIBUTE_UNUSED) { struct cl_option_state state; const char *data = (const char *)data_p; @@ -2221,7 +2221,6 @@ default_pch_valid_p (const void *data_p, size_t len) memcpy (&tf, data, sizeof (target_flags)); data += sizeof (target_flags); - len -= sizeof (target_flags); r = targetm.check_pch_target_flags (tf); if (r != NULL) return r; @@ -2233,7 +2232,6 @@ default_pch_valid_p (const void *data_p, size_t len) if (memcmp (data, state.data, state.size) != 0) return pch_option_mismatch (cl_options[i].opt_text); data += state.size; - len -= state.size; } return NULL; diff --git a/libcpp/charset.c b/libcpp/charset.c index b84a9740165..e4e45f6d39d 100644 --- a/libcpp/charset.c +++ b/libcpp/charset.c @@ -1464,7 +1464,6 @@ convert_oct (cpp_reader *pfile, const uchar *from, const uchar *limit, cppchar_t c, n = 0; size_t width = cvt.width; size_t mask = width_to_mask (width); - bool overflow = false; /* loc_reader and ranges must either be both NULL, or both be non-NULL. */ gcc_assert ((loc_reader != NULL) == (ranges != NULL)); @@ -1477,7 +1476,6 @@ convert_oct (cpp_reader *pfile, const uchar *from, const uchar *limit, from++; if (loc_reader) char_range.m_finish = loc_reader->get_next ().m_finish; - overflow |= n ^ (n << 3 >> 3); n = (n << 3) + c - '0'; }