On Tue, Jul 24, 2012 at 7:53 PM, Jason Merrill <ja...@redhat.com> wrote: > On 07/23/2012 01:01 PM, Steven Bosscher wrote: >> >> This large patch makes GATHER_STATISTICS always take a value, 0 >> (disabled) or 1 (enabled), and turns all related #ifdef code into >> conditional branches. > > > Did you test this with --enable-gather-detailed-mem-stats?
Yes, of course. > It didn't build > for me, so I fixed it with this change: Yes, the patch caused some more breakage elsewhere (--enable-checking=release, which I also tested). It turns out I had not updated the patch for the breakage I encountered. Sorry for the mess, and thanks for helping fix it! Ciao! Steven
