<div dir="ltr">This seems like a generally good thing to do if we're testing backtraces.<div><br></div><div>Do you want to merge the REQUIRES lines? I don't feel strongly either way.</div><div><br></div><div>Feel free to commit whenever.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 10, 2015 at 11:13 AM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Chandler, Owen<br>
<br>
In your patches to LLVM which changed ThreadLocal and PrettyStackTrace, we ended up requiring ENABLE_BACKTRACES for PrettyStackTrace.<br>
<br>
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.<br>
<br>
Thanks,<br>
Pete<br>
<br>
</blockquote></div><br></div>