I have submitted r201220 to fix this. I hope it is the right fix, at least it makes all the tests pass.
On Wed, Feb 12, 2014 at 9:08 AM, Dmitri Gribenko <[email protected]>wrote: > On Wed, Feb 12, 2014 at 7:55 AM, Chandler Carruth <[email protected]> > wrote: > > > > On Tue, Feb 11, 2014 at 11:26 PM, Daniel Jasper <[email protected]> > wrote: > >>> > >>> - OwnedBuffers.push_back(memBuf); > >> > >> > >> Not adding MemoryBuffers to OwnedBuffers creates a memory leak. Did you > >> intentionally remove that? > > > > This can't have been intentional. Introducing leaks isn't OK, I think its > > fine to either add back the necessary code or in a worst case given that > > this was just dead code cleanup, revert. Otherwise the leak-checking > build > > bots will stay red. =[ > > Hi Chandler, Daniel, > > Sorry for the breakage, I will fix this today. > > Dmitri > > -- > main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if > (j){printf("%d\n",i);}}} /*Dmitri Gribenko <[email protected]>*/ >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
