[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

2017-07-12 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL307856: Reland "[Driver] Update Fuchsia driver path handling" (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D35328?vs=106323&id=106327#toc Repository: rL LLVM https://revie

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

2017-07-12 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 106323. phosek marked an inline comment as done. Repository: rL LLVM https://reviews.llvm.org/D35328 Files: lib/Driver/ToolChains/Fuchsia.cpp lib/Driver/ToolChains/Fuchsia.h test/Driver/fuchsia.c test/Driver/fuchsia.cpp Index: test/Driver/fuchsia.c

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

2017-07-12 Thread Jonathan Roelofs via Phabricator via cfe-commits
jroelofs accepted this revision. jroelofs added inline comments. This revision is now accepted and ready to land. Comment at: lib/Driver/ToolChains/Fuchsia.cpp:134 +static std::string computeTriple(llvm::Triple Triple) { + SmallString<64> T; `normalizeTriple`

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

2017-07-12 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. I have fixed the test to also handle Windows paths properly which was causing a build breakage. Repository: rL LLVM https://reviews.llvm.org/D35328 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm

[PATCH] D35328: Reland "[Driver] Update Fuchsia driver path handling"

2017-07-12 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. Several improvements to the Fuchsia driver: - Search for C++ library headers and libraries in directories that are part of the toolchain distribution rather than sysroot. - Use LLVM support utlities to construct paths to make sure the driver is also usable on Wind