[PATCH] D13016: Check for GLIBC before including execinfo.h

Vasileios Kalintiris via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 22 03:33:56 PDT 2015

vkalintiris added a comment.

In http://reviews.llvm.org/D13016#250518, @brucem wrote:

> I missed this the other day. I think this change isn't needed because we can (I think) just not include `<execinfo.h>` now. It was added to support `Host::Backtrace` using `backtrace`, but that code has changed and is no longer here.

I tested it and it compiles fine without execinfo.h (Mint distro). Should I go on and remove the header? I'll keep an eye on the buildbots just in case.



More information about the llvm-commits mailing list