Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-07 Thread Jakub Jelinek
On Fri, Nov 07, 2014 at 10:01:45PM +0100, Richard Biener wrote: > > --- a/gcc/tree-ssa-tail-merge.c > > +++ b/gcc/tree-ssa-tail-merge.c > > @@ -484,7 +484,7 @@ same_succ_hash (const_same_succ e) > > > >hstate.add_int (gimple_code (stmt)); > >if (is_gimple_assign (stmt)) > > -

Re: [gimple-classes, committed 4/6] tree-ssa-tail-merge.c: Use gassign

2014-11-07 Thread Richard Biener
On Fri, Nov 7, 2014 at 4:21 PM, David Malcolm wrote: > gcc/ChangeLog.gimple-classes: > * tree-ssa-tail-merge.c (same_succ_hash): Add checked cast. > (gimple_equal_p): Add checked casts. > --- > gcc/ChangeLog.gimple-classes | 5 + > gcc/tree-ssa-tail-merge.c| 8 +--- >

Re: mt_allocator.cc assumes sizeof(size_t) == sizeof(void *)

2014-11-07 Thread Joel Sherrill
On 11/7/2014 9:25 AM, Paolo Carlini wrote: > Hi, > > On 11/07/2014 04:07 PM, Joel Sherrill wrote: >> Hi >> >> On m32c-rtems, we have a build error in C++ because size_t >> is 16-bits and pointers are 24 bits. m32c-elf probably does not >> enable __GTHREAD support like rtems does. Since this is co

Re: mt_allocator.cc assumes sizeof(size_t) == sizeof(void *)

2014-11-07 Thread Paolo Carlini
Hi, On 11/07/2014 04:07 PM, Joel Sherrill wrote: Hi On m32c-rtems, we have a build error in C++ because size_t is 16-bits and pointers are 24 bits. m32c-elf probably does not enable __GTHREAD support like rtems does. Since this is code shared across targets, what is the best way to fix this? I

Re: mt_allocator.cc assumes sizeof(size_t) == sizeof(void *)

2014-11-07 Thread Joel Sherrill
Attached is a patch which changes size_t to uintptr_t. It is enough to let the build continue. But I would appreciate feedback given the code. --joel On 11/7/2014 9:07 AM, Joel Sherrill wrote: > Hi > > On m32c-rtems, we have a build error in C++ because size_t > is 16-bits and pointers are 24 bits

mt_allocator.cc assumes sizeof(size_t) == sizeof(void *)

2014-11-07 Thread Joel Sherrill
Hi On m32c-rtems, we have a build error in C++ because size_t is 16-bits and pointers are 24 bits. m32c-elf probably does not enable __GTHREAD support like rtems does. Since this is code shared across targets, what is the best way to fix this? My first inclination would be to use uintptr_t for t

Re: Match-and-simplify and COND_EXPR

2014-11-07 Thread Richard Biener
On Thu, 6 Nov 2014, Andrew Pinski wrote: > On Thu, Nov 6, 2014 at 2:40 AM, Richard Biener wrote: > > On Thu, 6 Nov 2014, Richard Biener wrote: > > > >> On Wed, 5 Nov 2014, Andrew Pinski wrote: > >> > >> > Hi, > >> > I was trying to hook up tree-ssa-phiopt to match-and-simplify using > >> > eith

C++ headers vs safe-ctype.h

2014-11-07 Thread FX
Hi, We are currently seeing two cases of C++-related bootstrap failures on Darwin, due to C++ header being included after “system.h” and “safe-ctype.h”: headers, on Darwin, use ctype macros which GCC poisons through safe-ctype.h https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63703 https://gcc.

Re: Match-and-simplify and COND_EXPR

2014-11-07 Thread Richard Biener
On Thu, 6 Nov 2014, pins...@gmail.com wrote: > > > > > > On Nov 6, 2014, at 11:24 PM, Richard Biener wrote: > > > >> On November 7, 2014 5:03:19 AM CET, Andrew Pinski > >> wrote: > >> On Thu, Nov 6, 2014 at 2:40 AM, Richard Biener > >> wrote: > >>> On Thu, 6 Nov 2014, Richard Biener wrote