[PATCH] D56446: [Driver] Fix libcxx detection on Darwin with clang run as ./clang

2019-01-09 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added a comment. Sorry about the delay. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56446/new/ https://reviews.llvm.org/D56446 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm

[PATCH] D56446: [Driver] Fix libcxx detection on Darwin with clang run as ./clang

2019-01-09 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC350714: [Driver] Fix libcxx detection on Darwin with clang run as ./clang (authored by ibiryukov, committed by ). Changed prior to commit: https://reviews.llvm.org/D56446?vs=180691&id=180822#toc Reposi

[PATCH] D56446: [Driver] Fix libcxx detection on Darwin with clang run as ./clang

2019-01-09 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. …can you land this? :-) Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56446/new/ https://reviews.llvm.org/D56446 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/c

[PATCH] D56446: [Driver] Fix libcxx detection on Darwin with clang run as ./clang

2019-01-08 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman added a comment. This revision is now accepted and ready to land. That makes sense, LGTM. Thanks! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56446/new/ https://reviews.llvm.org/D56446 ___

[PATCH] D56446: [Driver] Fix libcxx detection on Darwin with clang run as ./clang

2019-01-08 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov created this revision. ilya-biryukov added reviewers: arphaman, thakis. Herald added a reviewer: EricWF. Herald added a subscriber: christof. By using '..' instead of fs::parent_path. The intention of the code was to go from 'path/to/clang/bin' to 'path/to/clang/include'. In most ca