<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Mar 1, 2013, at 3:35 PM, Jan Voung <<a href="mailto:jvoung@chromium.org">jvoung@chromium.org</a>> wrote:</div><blockquote type="cite"><div dir="ltr"><span style="font-family:arial,sans-serif;font-size:13px">Statistics are still available in Release+Asserts (or any +Asserts builds),</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">and they can also be turned on with LLVM_ENABLE_STATS.</span><br style="font-family:arial,sans-serif;font-size:13px">
<br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">Also change some of the FastISel stats that were moved under DEBUG()</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">back out of DEBUG(), since stats are disabled across the board now.</span><br style="font-family:arial,sans-serif;font-size:13px"><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">Many tests depend on grepping "-stats" output. Move those into</span><br style="font-family:arial,sans-serif;font-size:13px"><span style="font-family:arial,sans-serif;font-size:13px">a orig_dir/Stats/. so that they can be marked as unsupported</span><br style="font-family:arial,sans-serif;font-size:13px">
<span style="font-family:arial,sans-serif;font-size:13px">when building without statistics.</span></div></blockquote><br></div><div>Looks great to me! Please commit,</div><div><br></div><div>-Chris</div><br></body></html>