Hi,
On Wed, 23 May 2018, Richard Biener wrote:
> > alloc_flags (&cfun->cfg->bb_flag_pool);
> > alloc_flags (&cfun->cfg->edge_flag_pool);
>
> You'll end up with sth like
>
>alloc_flags flag (BB_FLAG_POOL_FOR_FN (cfun));
>
> then, mixing C++ RAII and macros! (eh)
First: I don't see
On Wed, 23 May 2018, Michael Matz wrote:
> Hi,
>
> On Wed, 23 May 2018, Eric Botcazou wrote:
>
> > > Maybe you should convert the thing to a template when the need arises
> > > instead of before? You have now added 54 lines of code for wrapping an
> > > int!
> >
> > Yeah, it took me 5 minutes
Hi,
On Wed, 23 May 2018, Eric Botcazou wrote:
> > Maybe you should convert the thing to a template when the need arises
> > instead of before? You have now added 54 lines of code for wrapping an
> > int!
>
> Yeah, it took me 5 minutes to understand what all this fluff is about!
So, what I thin
> Maybe you should convert the thing to a template when the need arises
> instead of before? You have now added 54 lines of code for wrapping an
> int!
Yeah, it took me 5 minutes to understand what all this fluff is about!
--
Eric Botcazou
Hi,
On Wed, 23 May 2018, Richard Biener wrote:
> > I messed up the conversion to a template. The bitnum should be
> > subtracted from HOST_BITS_PER_WIDE_INT and yes, 1 in unsigned hwi
> > should be shifted.
Maybe you should convert the thing to a template when the need arises
instead of befor
On Tue, 22 May 2018, Richard Biener wrote:
> On May 22, 2018 6:53:57 PM GMT+02:00, Joseph Myers
> wrote:
> >On Tue, 22 May 2018, Richard Biener wrote:
> >
> >> + if (*sptr & (1 << (CHAR_BIT * sizeof (T) - 1)))
> >> + gcc_unreachable ();
> >> + m_flag = 1 << ((CHAR_BIT * sizeof (T)) -
On May 22, 2018 6:53:57 PM GMT+02:00, Joseph Myers
wrote:
>On Tue, 22 May 2018, Richard Biener wrote:
>
>> + if (*sptr & (1 << (CHAR_BIT * sizeof (T) - 1)))
>> +gcc_unreachable ();
>> + m_flag = 1 << ((CHAR_BIT * sizeof (T)) - clz_hwi (*sptr));
>
>I don't see how the use of clz_hwi
On Tue, 22 May 2018, Richard Biener wrote:
> + if (*sptr & (1 << (CHAR_BIT * sizeof (T) - 1)))
> + gcc_unreachable ();
> + m_flag = 1 << ((CHAR_BIT * sizeof (T)) - clz_hwi (*sptr));
I don't see how the use of clz_hwi works with a type T that may be
narrower than HOST_WIDE_INT. Sur
On Tue, 22 May 2018, David Malcolm wrote:
> On Tue, 2018-05-22 at 10:43 +0200, Richard Biener wrote:
> > On Mon, 21 May 2018, Jeff Law wrote:
> >
> > > On 05/18/2018 07:15 AM, David Malcolm wrote:
> > > > On Fri, 2018-05-18 at 13:11 +0200, Richard Biener wrote:
> > > > > The following adds a simp
On Tue, 2018-05-22 at 10:43 +0200, Richard Biener wrote:
> On Mon, 21 May 2018, Jeff Law wrote:
>
> > On 05/18/2018 07:15 AM, David Malcolm wrote:
> > > On Fri, 2018-05-18 at 13:11 +0200, Richard Biener wrote:
> > > > The following adds a simple alloc/free_flag machinery
> > > > allocating
> > > >
On Mon, 21 May 2018, Jeff Law wrote:
> On 05/18/2018 07:15 AM, David Malcolm wrote:
> > On Fri, 2018-05-18 at 13:11 +0200, Richard Biener wrote:
> >> The following adds a simple alloc/free_flag machinery allocating
> >> bits from an int typed pool and applies that to bb->flags and edge-
> >>> flag
On 05/18/2018 07:15 AM, David Malcolm wrote:
> On Fri, 2018-05-18 at 13:11 +0200, Richard Biener wrote:
>> The following adds a simple alloc/free_flag machinery allocating
>> bits from an int typed pool and applies that to bb->flags and edge-
>>> flags.
>> This should allow infrastructure pieces to
On Fri, 2018-05-18 at 13:11 +0200, Richard Biener wrote:
> The following adds a simple alloc/free_flag machinery allocating
> bits from an int typed pool and applies that to bb->flags and edge-
> >flags.
> This should allow infrastructure pieces to use egde/bb flags
> temporarily
> without worrying
13 matches
Mail list logo
