Doh, I think I found an issue with this patch :-/ I'm building a reduced test case now, I'll post it here in a few hours.
Nico On Tue, Feb 21, 2012 at 5:57 PM, Eric Christopher <[email protected]> wrote: > > On Feb 19, 2012, at 7:11 PM, Rafael Ávila de Espíndola > <[email protected]> wrote: > >> Nico noticed that the patch included an infinite loop from when I >> renamed merge2 to mergeWithMin. >> >> An updated patch is attached. > > I think this is ok. Some more comments on the visibility that should be > happening in the various testcases and where you use mergeWithMin would be > helpful next time I need to review something through there. > > -eric _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
