Clean up fallout on ILP32 from r229831.

gcc/
	PR c++/67942
	* cp/init.c (warn_placement_new_too_small): Convert integer
	operand of POINTER_PLUS_EXPR to ssize_t to determine its signed
	value.

c-family/
        * c.opt (Wplacement-new): Add a period to the end of
        a sentence.

From-SVN: r229959

diff --git a/gcc/c-family/ChangeLog b/gcc/c-family/ChangeLog
index 7a2781d3c7efafa8fb6388e725b7c8e16abedf2a..1b296ca364e0333008e19716a94417dcee77b32b 100644 (file)
--- a/gcc/c-family/ChangeLog
+++ b/gcc/c-family/ChangeLog
@@ -1,3 +1,7 @@
+2015-11-08  Martin Sebor  <msebor@redhat.com>
+
+       * c.opt (Wplacement-new): Add a period to the end of a sentence.
+
 2015-11-07  Richard Sandiford  <richard.sandiford@arm.com>
 
        * c-common.c: Don't undef DEF_BUILTIN.

diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt
index 5f38018d35e917b1664c7f6f6b531f0860be0332..f066400c6c5b61d2795c71a8aca48105f17e64d5 100644 (file)
--- a/gcc/c-family/c.opt
+++ b/gcc/c-family/c.opt
@@ -778,7 +778,7 @@ Warn if inherited methods are unimplemented.
 
 Wplacement-new
 C++ Var(warn_placement_new) Init(1) Warning
-Warn for placement new expressions with undefined behavior
+Warn for placement new expressions with undefined behavior.
 
 Wredundant-decls
 C ObjC C++ ObjC++ Var(warn_redundant_decls) Warning

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index fc92a5f0561c05e37bd2f3c538bf338a5263c62b..11ca9ab584a8845572ee48fba969b55a64cf1744 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,10 @@
+2015-11-08  Martin Sebor  <msebor@redhat.com>
+
+       PR c++/67942 
+       * cp/init.c (warn_placement_new_too_small): Convert integer
+       operand of POINTER_PLUS_EXPR to ssize_t to determine its signed
+       value.
+
 2015-11-06  David Malcolm  <dmalcolm@redhat.com>
 
        * error.c (cp_printer): Update for new "caret_p" param for

diff --git a/gcc/cp/init.c b/gcc/cp/init.c
index 337797c986ba94bab27b5d1ec73b6d7c658f6cdd..b45281f517db704ff11c8806380e71af4c89adb0 100644 (file)
--- a/gcc/cp/init.c
+++ b/gcc/cp/init.c
@@ -2321,12 +2321,14 @@ warn_placement_new_too_small (tree type, tree nelts, tree size, tree oper)
   if (TREE_CODE (oper) == POINTER_PLUS_EXPR)
     {
       /* If the offset is comple-time constant, use it to compute a more
-        accurate estimate of the size of the buffer.  Otherwise, use
-        the size of the entire array as an optimistic estimate (this
-        may lead to false negatives).  */
-      const_tree adj = TREE_OPERAND (oper, 1);
+        accurate estimate of the size of the buffer.  Since the operand
+        of POINTER_PLUS_EXPR is represented as an unsigned type, convert
+        it to signed first.
+        Otherwise, use the size of the entire array as an optimistic
+        estimate (this may lead to false negatives).  */
+      tree adj = TREE_OPERAND (oper, 1);
       if (CONSTANT_CLASS_P (adj))
-       adjust += tree_to_uhwi (adj);
+       adjust += tree_to_shwi (convert (ssizetype, adj));
       else
        use_obj_size = true;