[PATCH] Teach llvm-lit about ENABLE_BACKTRACES

Chandler Carruth chandlerc at google.com
Tue Feb 10 11:50:58 PST 2015


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 <peter_cooper at apple.com>
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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150210/01b7a302/attachment.html>


More information about the cfe-commits mailing list