[RFC PATCH] add auto_bitmap

Trevor Saunders tsaunders@mozilla.com
Fri Nov 15 11:35:00 GMT 2013


On Fri, Nov 15, 2013 at 10:56:24AM +0100, Richard Biener wrote:
> On Thu, Nov 14, 2013 at 10:33 PM, Jeff Law <law@redhat.com> wrote:
> > On 11/14/13 14:14, Richard Biener wrote:
> >>>
> >>>
> >>> I'm just pointed out that of all the stuff you changed, these were the
> >>> only ones I saw where lifetimes were changed significantly.
> >>
> >>
> >> I still ask why we need a new type and cannot put this functionality into
> >> bitmap_head itself.
> >
> > Given that bitmap is just a *bitmap_head_def aren't we suggesting the same
> > thing?
> 
> Not sure - I thought Trevor wanted to make auto_bitmap a full C++ thing,
> not bitmap itself?

My only firm goals are less manual memory management, and moving the
bitmap_head bit onto the stack would be really nice.  I'd also like to
leave bitmaps allocated in gc memory alone for the time being, but those
are the only firm goals.  I'm currently trying the approach of adding
constructors and destructors to bitmap_head, but apparently something is
causing them to get invoked even when everybody deals with bitmap_head *
which leads to ICEs that I'm investigating now.

Trev

> 
> Richard.
> 
> > jeff



More information about the Gcc-patches mailing list