[PATCH] D62442: [Driver] Update handling of c++ and runtime directories

2019-05-25 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC361709: [Driver] Update handling of c++ and runtime directories (authored by phosek, committed by ). Changed prior to commit: https://reviews.llvm.org/D62442?vs=201432&id=201433#toc Repository: rC Cl

[PATCH] D62442: [Driver] Update handling of c++ and runtime directories

2019-05-25 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D62442#1517183 , @mcgrathr wrote: > lgtm contingent on verifying intended behavior changes and adding comments. > The factoring suggestion for the triple,triple searching is preferred but at > your discretion, though I'd reall

[PATCH] D62442: [Driver] Update handling of c++ and runtime directories

2019-05-25 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 201432. phosek marked 8 inline comments as done. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62442/new/ https://reviews.llvm.org/D62442 Files: clang/include/clang/Driver/ToolChain.h clang/lib/Driver/ToolChain.cpp clang/

[PATCH] D62442: [Driver] Update handling of c++ and runtime directories

2019-05-24 Thread Roland McGrath via Phabricator via cfe-commits
mcgrathr accepted this revision. mcgrathr added a comment. This revision is now accepted and ready to land. lgtm contingent on verifying intended behavior changes and adding comments. The factoring suggestion for the triple,triple searching is preferred but at your discretion, though I'd really

[PATCH] D62442: [Driver] Update handling of c++ and runtime directories

2019-05-24 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added reviewers: mcgrathr, leonardchan, saugustine, echristo. Herald added subscribers: cfe-commits, javed.absar, kubamracek. Herald added a project: clang. This is a follow up to r361432 and r361504 which addresses issues introduced by those changes. Specifica