Re: [C++ PATCH] -Wsizeof-pointer-memaccess warning (take 2)

2012-10-26 Thread Roman Zhuykov
This caused http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55081 -- Roman Zhuykov

Committed, PR54897 (was: [C++ PATCH] -Wsizeof-pointer-memaccess warning (take 2))

2012-10-11 Thread Hans-Peter Nilsson
From: Hans-Peter Nilsson h...@axis.com Date: Thu, 11 Oct 2012 02:13:32 +0200 There's now an excess error: x/libstdc++-v3/testsuite/23_containers/bitset/45713.cc:24:55: error: size of array 'test' is not an integral constant-expression int test[sizeof(std::bitset0x) != 1 ? 1 :

Re: [C++ PATCH] -Wsizeof-pointer-memaccess warning (take 2)

2012-10-10 Thread Hans-Peter Nilsson
From: Jakub Jelinek ja...@redhat.com Date: Tue, 2 Oct 2012 14:56:29 +0200 2012-10-02 Jakub Jelinek ja...@redhat.com cp/ * cp-tree.h (SIZEOF_EXPR_TYPE_P): Define. * tree.c (cp_tree_equal): Handle SIZEOF_EXPR with SIZEOF_EXPR_TYPE_P. ...etc. Looks like this

Re: [C++ PATCH] -Wsizeof-pointer-memaccess warning (take 2)

2012-10-05 Thread Jason Merrill
On 10/02/2012 08:56 AM, Jakub Jelinek wrote: Perhaps maybe_constant_value already does all that integral_constant_value does, but I'm not sure about it. It does, you can remove that call. OK with that change. Jason

Re: [C++ PATCH] -Wsizeof-pointer-memaccess warning (take 2)

2012-10-03 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com a écrit: --- gcc/cp/call.c.jj 2012-09-27 12:45:49.0 +0200 +++ gcc/cp/call.c 2012-10-01 17:53:17.594609236 +0200 @@ -557,7 +557,10 @@ null_ptr_cst_p (tree t) { /* Core issue 903 says only literal 0 is a null pointer constant. */

Re: [C++ PATCH] -Wsizeof-pointer-memaccess warning (take 2)

2012-10-03 Thread Jakub Jelinek
On Wed, Oct 03, 2012 at 11:14:37AM +0200, Dodji Seketeli wrote: Jakub Jelinek ja...@redhat.com a écrit: --- gcc/cp/call.c.jj2012-09-27 12:45:49.0 +0200 +++ gcc/cp/call.c 2012-10-01 17:53:17.594609236 +0200 @@ -557,7 +557,10 @@ null_ptr_cst_p (tree t) { /*