Thanks for the review Chris! The original LLVM patch still applies, but I'd missed the fact that a few Clang tests also depend on Statistic.h.
+cfe-commits Here is a follow on patch for Clang that moves 2 tests that use "-analyzer-stats", which I think depends on stats. On Fri, Mar 8, 2013 at 1:15 PM, Chris Lattner <[email protected]> wrote: > On Mar 1, 2013, at 3:35 PM, Jan Voung <[email protected]> wrote: > > Statistics are still available in Release+Asserts (or any +Asserts builds), > and they can also be turned on with LLVM_ENABLE_STATS. > > Also change some of the FastISel stats that were moved under DEBUG() > back out of DEBUG(), since stats are disabled across the board now. > > Many tests depend on grepping "-stats" output. Move those into > a orig_dir/Stats/. so that they can be marked as unsupported > when building without statistics. > > > Looks great to me! Please commit, > > -Chris > >
0001-Move-clang-tests-that-depend-on-llvm-ADT-Statistic.h.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
