diff options
| author | Richard Biener <rguenther@suse.de> | 2017-03-17 11:04:00 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2017-03-17 11:04:00 +0000 |
| commit | d59b72533ffbe4e11ef2bb12cfaffece3af58c74 (patch) | |
| tree | 553a07a6c9270d33bcf9215373daafd090e70d3c /gcc | |
| parent | 61c81999bfd6edc4fa414bba2acdfa502fbf07d4 (diff) | |
| download | gcc-d59b72533ffbe4e11ef2bb12cfaffece3af58c74.zip gcc-d59b72533ffbe4e11ef2bb12cfaffece3af58c74.tar.gz gcc-d59b72533ffbe4e11ef2bb12cfaffece3af58c74.tar.bz2 | |
re PR other/80050 (gcc/genmatch.c: PVS-Studio: V590)
2017-03-17 Richard Biener <rguenther@suse.de>
PR middle-end/80050
* genmatch.c (parser::next): Remove pointless check for CPP_EOF.
(parser::peek): Likewise.
From-SVN: r246218
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/genmatch.c | 6 |
2 files changed, 8 insertions, 4 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 925709b..9a160fa 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,11 @@ 2017-03-17 Richard Biener <rguenther@suse.de> + PR middle-end/80050 + * genmatch.c (parser::next): Remove pointless check for CPP_EOF. + (parser::peek): Likewise. + +2017-03-17 Richard Biener <rguenther@suse.de> + PR tree-optimization/80048 * sese.c (free_sese_info): Properly release rename_map and copied_bb_map elements. diff --git a/gcc/genmatch.c b/gcc/genmatch.c index fdf5aaf..93d5b21 100644 --- a/gcc/genmatch.c +++ b/gcc/genmatch.c @@ -3826,8 +3826,7 @@ parser::next () { token = cpp_get_token (r); } - while (token->type == CPP_PADDING - && token->type != CPP_EOF); + while (token->type == CPP_PADDING); return token; } @@ -3842,8 +3841,7 @@ parser::peek (unsigned num) { token = cpp_peek_token (r, i++); } - while ((token->type == CPP_PADDING - && token->type != CPP_EOF) + while (token->type == CPP_PADDING || (--num > 0)); /* If we peek at EOF this is a fatal error as it leaves the cpp_reader in unusable state. Assume we really wanted a |