<div dir="ltr">Thanks for the review Chris!<div><br></div><div style>The original LLVM patch still applies, but I'd missed the fact that a few Clang tests also depend on Statistic.h.</div><div style><br></div><div style>
+cfe-commits</div><div style><br></div><div style>Here is a follow on patch for Clang that moves 2 tests that use "-analyzer-stats", which I think depends on stats.</div><div style><br></div><div style><br></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 8, 2013 at 1:15 PM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com" target="_blank">clattner@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 class="h5"><div><div>On Mar 1, 2013, at 3:35 PM, Jan Voung <<a href="mailto:jvoung@chromium.org" target="_blank">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></div><div>Looks great to me!  Please commit,</div><div><br></div><div>-Chris</div><br>
</div>
</blockquote></div><br></div>