[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