From: Richard Biener <rguenther@suse.de>
Date: Mon, 14 Jul 2014 11:12:29 +0000 (+0000)
Subject: builtins.c (c_strlen): Make only_value == 2 really only affect warning generation.
X-Git-Tag: releases/gcc-5.1.0~6352
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=3b57ff81e6246551d6eb563ff2981027db9e6205;p=thirdparty%2Fgcc.git

builtins.c (c_strlen): Make only_value == 2 really only affect warning generation.

2014-07-14  Richard Biener  <rguenther@suse.de>

	* builtins.c (c_strlen): Make only_value == 2 really only
	affect warning generation.

From-SVN: r212514
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 43f871fce816..04cece4ee76b 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2014-07-14  Richard Biener  <rguenther@suse.de>
+
+	* builtins.c (c_strlen): Make only_value == 2 really only
+	affect warning generation.
+
 2014-07-14  Richard Biener  <rguenther@suse.de>
 
 	PR tree-optimization/61757
diff --git a/gcc/builtins.c b/gcc/builtins.c
index cc94302d0cf5..03d1dcadfa13 100644
--- a/gcc/builtins.c
+++ b/gcc/builtins.c
@@ -610,11 +610,11 @@ c_strlen (tree src, int only_value)
 
   /* If the offset is known to be out of bounds, warn, and call strlen at
      runtime.  */
-  if (only_value != 2
-      && (offset < 0 || offset > max))
+  if (offset < 0 || offset > max)
     {
      /* Suppress multiple warnings for propagated constant strings.  */
-      if (! TREE_NO_WARNING (src))
+      if (only_value != 2
+	  && !TREE_NO_WARNING (src))
         {
           warning_at (loc, 0, "offset outside bounds of constant string");
           TREE_NO_WARNING (src) = 1;