This seems like a generally good thing to do if we're testing backtraces. Do you want to merge the REQUIRES lines? I don't feel strongly either way.
Feel free to commit whenever. On Tue, Feb 10, 2015 at 11:13 AM, Pete Cooper <[email protected]> wrote: > Hi Chandler, Owen > > In your patches to LLVM which changed ThreadLocal and PrettyStackTrace, we > ended up requiring ENABLE_BACKTRACES for PrettyStackTrace. > > Trouble is, clang has 2 tests which check for the pretty stack trace on > crashes. This patch teaches lit about the ENABLE_BACKTRACES option, and > updates those 2 tests to use that option. Without this patch, clang fails > to pass llvm-lit without back traces. > > Thanks, > Pete > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
