Re: [RFA jit 2/2] introduce scoped_timevar

2014-03-19 Thread Tom Tromey
Trevor == Trevor Saunders tsaund...@mozilla.com writes: Trevor thanks for doing this. I wonder about naming, we already have Trevor auto_vec and while I don't really care wether we use auto_ or Trevor scoped_ it seems like being consistant would be nice. Sounds reasonable to me, I've made this

[RFA jit 2/2] introduce scoped_timevar

2014-03-18 Thread Tom Tromey
This introduces a new scoped_timevar class. It pushes a given timevar in its constructor, and pops it in the destructor, giving a much simpler way to use timevars in the typical case where they can be scoped. --- gcc/ChangeLog.jit | 4 gcc/jit/ChangeLog.jit | 4

Re: [RFA jit 2/2] introduce scoped_timevar

2014-03-18 Thread David Malcolm
On Tue, 2014-03-18 at 08:57 -0600, Tom Tromey wrote: This introduces a new scoped_timevar class. It pushes a given timevar in its constructor, and pops it in the destructor, giving a much simpler way to use timevars in the typical case where they can be scoped. --- gcc/ChangeLog.jit |

Re: [RFA jit 2/2] introduce scoped_timevar

2014-03-18 Thread Trevor Saunders
On Tue, Mar 18, 2014 at 08:57:44AM -0600, Tom Tromey wrote: This introduces a new scoped_timevar class. It pushes a given timevar in its constructor, and pops it in the destructor, giving a much simpler way to use timevars in the typical case where they can be scoped. thanks for doing this.