Re: [PATCH] use stack vectors more

2013-11-01 Thread Diego Novillo
On Fri, Nov 1, 2013 at 3:51 PM, Diego Novillo wrote: > It must've been whitespace then. Your new patch applied just fine. > I'll be committing shortly. Committed at r204301. Diego.

Re: [PATCH] use stack vectors more

2013-11-01 Thread Diego Novillo
On Fri, Nov 1, 2013 at 3:33 PM, Trevor Saunders wrote: >> The patch is OK, but it did not completely apply in my tree. Mind >> sending an updated version (or point me at a git repo I can pull it >> from). > > interesting, I just pulled and rebased it onto r204296 without any manual > merging. Pat

Re: [PATCH] use stack vectors more

2013-11-01 Thread Trevor Saunders
> The patch is OK, but it did not completely apply in my tree. Mind > sending an updated version (or point me at a git repo I can pull it > from). interesting, I just pulled and rebased it onto r204296 without any manual merging. Patch against r204296 attached (obviously haven't tested it against

Re: [PATCH] use stack vectors more

2013-11-01 Thread Diego Novillo
On Thu, Oct 31, 2013 at 7:48 PM, wrote: > From: Trevor Saunders > > Hi, > > This patch is pretty dull, it just replaces a bunch of things of the form > vec x; > x.create (N); // N is a constant > blah blah > x.release (); > by > stack_vec x; > blah blah > > Of course its even nicer than that in