<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Feb 28, 2013 at 9:13 AM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div>Hi Eric, </div><div class="im"><div><br></div><blockquote type="cite"><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
This is what I was talking about that Nadav never bothered to respond to.</div></blockquote><div><br></div></div><div>You wrote me less than 24 hours ago, and I did not have time to write back. </div><div class="im"><br><blockquote type="cite">
<div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
</div></blockquote></div></div></div></blockquote><div><br></div><div style>Yeah, sorry about that I was just responding to myself and the list about that.</div><div style>My apologies.</div><div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="im"><blockquote type="cite"><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
Whether or not it made sense to put all of the statistics as only firing during</div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
debug builds and then making the change in the statistics framework rather</div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
than #if NDEBUG for each set of statistics and the assorted increments.</div><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
<br></div></blockquote><div><br></div></div><div>I don't have any preference. Do you prefer one way or the other ?</div><div><br></div></div><div>I kept NumDAGBlocks and NumFastIselBlocks, because they are called once per blocks and not once per instructions. I don't mind if we put them behind #ifndef NDEBUG as well.  </div>
</div></blockquote><div><br></div><div style>It's why I was asking, I think it might be an interesting idea to put all the statistics</div><div style>behind NDEBUG so we can feel free to collect stats whenever rather than</div>
<div style>needing to worry about putting the high firing status under NDEBUG explicitly.</div><div style><br></div><div style>-eric</div></div></div></div>