Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-20 Thread Richard Biener
On Thu, 20 Apr 2017, Jakub Jelinek wrote: > On Wed, Apr 19, 2017 at 08:22:23AM +0200, Jakub Jelinek wrote: > > Here in (so far untested) patch form: > > > > 2017-04-19 Jakub Jelinek > > > > PR middle-end/80423 > > * tree.h (build_array_type): Add typeless_storage default argument. > >

Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-19 Thread Jakub Jelinek
On Wed, Apr 19, 2017 at 08:22:23AM +0200, Jakub Jelinek wrote: > Here in (so far untested) patch form: > > 2017-04-19 Jakub Jelinek > > PR middle-end/80423 > * tree.h (build_array_type): Add typeless_storage default argument. > * tree.c (type_cache_hasher::equal): Also compar

Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-19 Thread Richard Biener
On April 19, 2017 7:56:30 AM GMT+02:00, Jakub Jelinek wrote: >On Wed, Apr 19, 2017 at 07:45:36AM +0200, Richard Biener wrote: >> >As mentioned in the PR, we now use TYPE_TYPELESS_STORAGE flag on >> >ARRAY_TYPEs to denote types that need the special C++ alias >handling. >> >The problem is how is th

Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-18 Thread Jakub Jelinek
On Wed, Apr 19, 2017 at 07:56:30AM +0200, Jakub Jelinek wrote: > On Wed, Apr 19, 2017 at 07:45:36AM +0200, Richard Biener wrote: > > >As mentioned in the PR, we now use TYPE_TYPELESS_STORAGE flag on > > >ARRAY_TYPEs to denote types that need the special C++ alias handling. > > >The problem is how i

Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-18 Thread Jakub Jelinek
On Wed, Apr 19, 2017 at 07:45:36AM +0200, Richard Biener wrote: > >As mentioned in the PR, we now use TYPE_TYPELESS_STORAGE flag on > >ARRAY_TYPEs to denote types that need the special C++ alias handling. > >The problem is how is that created, we just use build_array_type and > >set TYPE_TYPELESS_S

Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-18 Thread Richard Biener
On April 18, 2017 5:14:30 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >As mentioned in the PR, we now use TYPE_TYPELESS_STORAGE flag on >ARRAY_TYPEs to denote types that need the special C++ alias handling. >The problem is how is that created, we just use build_array_type and >set TYPE_TYPELESS_STOR

Re: [PATCH] Fix TYPE_TYPELESS_STORAGE handling (PR middle-end/80423)

2017-04-18 Thread Jeff Law
On 04/18/2017 09:14 AM, Jakub Jelinek wrote: Hi! As mentioned in the PR, we now use TYPE_TYPELESS_STORAGE flag on ARRAY_TYPEs to denote types that need the special C++ alias handling. The problem is how is that created, we just use build_array_type and set TYPE_TYPELESS_STORAGE on the result, bu