[Lldb-commits] [PATCH] D25487: Fix building tests without system headers on Darwin

2016-10-13 Thread Todd Fiala via lldb-commits
tfiala added a comment. (Retro) yep, this looked fine. Repository: rL LLVM https://reviews.llvm.org/D25487 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D25487: Fix building tests without system headers on Darwin

2016-10-12 Thread Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL284042: Fix building tests without system headers on Darwin (authored by cbieneman). Changed prior to commit: https://reviews.llvm.org/D25487?vs=74293&id=74430#toc Repository: rL LLVM https://review

[Lldb-commits] [PATCH] D25487: Fix building tests without system headers on Darwin

2016-10-11 Thread Zachary Turner via lldb-commits
zturner accepted this revision. zturner added a comment. This revision is now accepted and ready to land. If you wanted to get really fancy, you could actually locate the system include directory and set the default to that. But that's up to you, just an idea. https://reviews.llvm.org/D25487

[Lldb-commits] [PATCH] D25487: Fix building tests without system headers on Darwin

2016-10-11 Thread Chris Bieneman via lldb-commits
beanz created this revision. beanz added reviewers: tfiala, zturner. beanz added a subscriber: lldb-commits. Default installations of OS X do not have system headers installed at /usr/include. This patch allows the LLDB test executables to properly compile when built on a system without headers