From: Jason Merrill <jason@redhat.com>
Date: Wed, 3 Feb 2010 16:28:07 +0000 (-0500)
Subject: re PR c++/35652 (offset warning should be given in the front-end)
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=59d49708b2f28dfd32e760dbdabe8b97ba5504e1;p=gcc.git

re PR c++/35652 (offset warning should be given in the front-end)

	PR c++/35652
	* builtins.c (c_strlen): Use EXPR_LOCATION in diagnostics.

From-SVN: r156469
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 180d60431a1..8e5b7bc75f6 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2010-02-03  Jason Merrill  <jason@redhat.com>
+
+	PR c++/35652
+	* builtins.c (c_strlen): Use EXPR_LOCATION in diagnostics.
+
 2010-02-03  Alexandre Oliva  <aoliva@redhat.com>
 
 	PR debug/42896
diff --git a/gcc/builtins.c b/gcc/builtins.c
index ac9693400e4..b6fd1ead0fb 100644
--- a/gcc/builtins.c
+++ b/gcc/builtins.c
@@ -433,6 +433,7 @@ c_strlen (tree src, int only_value)
   HOST_WIDE_INT offset;
   int max;
   const char *ptr;
+  location_t loc;
 
   STRIP_NOPS (src);
   if (TREE_CODE (src) == COND_EXPR
@@ -450,6 +451,11 @@ c_strlen (tree src, int only_value)
       && (only_value || !TREE_SIDE_EFFECTS (TREE_OPERAND (src, 0))))
     return c_strlen (TREE_OPERAND (src, 1), only_value);
 
+  if (EXPR_HAS_LOCATION (src))
+    loc = EXPR_LOCATION (src);
+  else
+    loc = input_location;
+
   src = string_constant (src, &offset_node);
   if (src == 0)
     return NULL_TREE;
@@ -475,7 +481,7 @@ c_strlen (tree src, int only_value)
 	 and return that.  This would perhaps not be valid if we were dealing
 	 with named arrays in addition to literal string constants.  */
 
-      return size_diffop_loc (input_location, size_int (max), offset_node);
+      return size_diffop_loc (loc, size_int (max), offset_node);
     }
 
   /* We have a known offset into the string.  Start searching there for
@@ -494,7 +500,7 @@ c_strlen (tree src, int only_value)
      /* Suppress multiple warnings for propagated constant strings.  */
       if (! TREE_NO_WARNING (src))
         {
-          warning (0, "offset outside bounds of constant string");
+          warning_at (loc, 0, "offset outside bounds of constant string");
           TREE_NO_WARNING (src) = 1;
         }
       return NULL_TREE;
diff --git a/gcc/testsuite/g++.dg/warn/string1.C b/gcc/testsuite/g++.dg/warn/string1.C
new file mode 100644
index 00000000000..302772795cd
--- /dev/null
+++ b/gcc/testsuite/g++.dg/warn/string1.C
@@ -0,0 +1,18 @@
+// PR c++/35652
+// { dg-options "-O" }
+
+#include <string>
+int main() {
+  // blank line padding, could also be code...
+  //
+  //
+  //
+  //
+  //
+  //
+  //
+  //
+  //
+  std::string s = "";
+  s += 'x' + "y";	      // { dg-warning "bounds of constant string" }
+}