This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/6] add auto_sbitmap class

From: Trevor Saunders <tbsaunde at tbsaunde dot org>
To: Jeff Law <law at redhat dot com>
Cc: Richard Biener <richard dot guenther at gmail dot com>, tbsaunde+gcc at tbsaunde dot org, gcc-patches at gcc dot gnu dot org
Date: Mon, 25 Jul 2016 22:01:35 -0400
Subject: Re: [PATCH 1/6] add auto_sbitmap class
Authentication-results: sourceware.org; auth=none
References: <20160724114449.12008-1-tbsaunde+gcc@tbsaunde.org> <20160724114449.12008-2-tbsaunde+gcc@tbsaunde.org> <DEDBC2DF-B3CD-43DE-9111-6CC8379DD739@gmail.com> <89a7decf-ab6b-645f-53c3-c001993e0041@redhat.com>

On Mon, Jul 25, 2016 at 09:30:07AM -0600, Jeff Law wrote:
> On 07/24/2016 10:28 AM, Richard Biener wrote:
> > On July 24, 2016 1:44:44 PM GMT+02:00, tbsaunde+gcc@tbsaunde.org wrote:
> > > From: Trevor Saunders <tbsaunde+gcc@tbsaunde.org>
> > > 
> > > gcc/ChangeLog:
> > > 
> > > 2016-07-24  Trevor Saunders  <tbsaunde+gcc@tbsaunde.org>
> > > 
> > > 	* sbitmap.h (auto_sbitmap): New class.
> > 
> > OK.
> Do we need a private move constructor and assignment operator here too?

I think they aren't generated by default, but it shouldn't hurt, and we
should probably add a compat macro for = delete, I'll try to get to that
part soon.

Trev

> 
> 
> jeff
>

References:
- [PATCH 0/6] add and use an auto_sbitmap class
  - From: tbsaunde+gcc
- [PATCH 1/6] add auto_sbitmap class
  - From: tbsaunde+gcc
- Re: [PATCH 1/6] add auto_sbitmap class
  - From: Richard Biener
- Re: [PATCH 1/6] add auto_sbitmap class
  - From: Jeff Law

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]