This revision was automatically updated to reflect the committed changes.
Closed by commit rC342021: [Driver] Search LibraryPaths when handling
-print-file-name (authored by phosek, committed by ).
Changed prior to commit:
https://reviews.llvm.org/D51573?vs=163636&id=165008#toc
Repository:
r
phosek added inline comments.
Comment at: clang/lib/Driver/Driver.cpp:4187
SmallString<128> R(ResourceDir);
llvm::sys::path::append(R, Name);
mcgrathr wrote:
> You could fold these cases into the lambda too by having it take a bool
> UseSysRoot and passi
phosek updated this revision to Diff 163636.
phosek marked 4 inline comments as done.
Repository:
rC Clang
https://reviews.llvm.org/D51573
Files:
clang/lib/Driver/Driver.cpp
clang/test/Driver/linux-per-target-runtime-dir.c
Index: clang/test/Driver/linux-per-target-runtime-dir.c
=
mcgrathr accepted this revision.
mcgrathr added a comment.
This revision is now accepted and ready to land.
The log message could give concrete examples.
Comment at: clang/lib/Driver/Driver.cpp:4169
std::string Driver::GetFilePath(StringRef Name, const ToolChain &TC) const {
-
phosek updated this revision to Diff 163633.
Repository:
rC Clang
https://reviews.llvm.org/D51573
Files:
clang/lib/Driver/Driver.cpp
clang/test/Driver/linux-per-target-runtime-dir.c
Index: clang/test/Driver/linux-per-target-runtime-dir.c
==
phosek updated this revision to Diff 163630.
Repository:
rC Clang
https://reviews.llvm.org/D51573
Files:
clang/lib/Driver/Driver.cpp
Index: clang/lib/Driver/Driver.cpp
===
--- clang/lib/Driver/Driver.cpp
+++ clang/lib/Driver/D
phosek created this revision.
phosek added a reviewer: beanz.
Herald added a subscriber: cfe-commits.
This is necessary to handle the multiarch runtime directories.
Repository:
rC Clang
https://reviews.llvm.org/D51573
Files:
clang/lib/Driver/Driver.cpp
Index: clang/lib/Driver/Driver.cpp