From eff89e01b880e6f058ae136935d70c743bc852b6 Mon Sep 17 00:00:00 2001 From: Bernd Edlinger Date: Thu, 20 Oct 2016 14:00:46 +0000 Subject: [PATCH] c-common.c (c_common_truthvalue_conversion): Fix the comment. 2016-10-20 Bernd Edlinger * c-common.c (c_common_truthvalue_conversion): Fix the comment. From-SVN: r241377 --- gcc/c-family/ChangeLog | 4 ++++ gcc/c-family/c-common.c | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog index ae399ea3142..b16b1c1fac4 100644 --- a/gcc/c-family/ChangeLog +++ b/gcc/c-family/ChangeLog @@ -1,3 +1,7 @@ +2016-10-20 Bernd Edlinger + + * c-common.c (c_common_truthvalue_conversion): Fix the comment. + 2016-10-20 Jason Merrill * c-cppbuiltin.c (c_cpp_builtins): Update __cpp_concepts value. diff --git a/gcc/c-family/c-common.c b/gcc/c-family/c-common.c index 8af3ad53a3d..abc07402640 100644 --- a/gcc/c-family/c-common.c +++ b/gcc/c-family/c-common.c @@ -3328,7 +3328,7 @@ c_common_truthvalue_conversion (location_t location, tree expr) TREE_OPERAND (expr, 0)); case LSHIFT_EXPR: - /* We will only warn on unsigned shifts here, because the majority of + /* We will only warn on signed shifts here, because the majority of false positive warnings happen in code where unsigned arithmetic was used in anticipation of a possible overflow. Furthermore, if we see an unsigned type here we know that the -- 2.30.2