[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