keith abandoned this revision.
keith added a comment.
https://reviews.llvm.org/D83015
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80660/new/
https://reviews.llvm.org/D80660
___
cfe-commits mailing
keith updated this revision to Diff 267991.
keith added a comment.
Fix test on windows
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80660/new/
https://reviews.llvm.org/D80660
Files:
clang/lib/Driver/ToolChain.cpp
clang/test/Driver/fuse-ld.c
keith added a comment.
The new test failure appears unrelated to my changes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80660/new/
https://reviews.llvm.org/D80660
___
cfe-commits mailing list
keith added a comment.
Thanks I moved them to the bottom and I believe I fixed them on windows!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80660/new/
https://reviews.llvm.org/D80660
___
cfe-commits
keith updated this revision to Diff 267069.
keith added a comment.
Update tests
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80660/new/
https://reviews.llvm.org/D80660
Files:
clang/lib/Driver/ToolChain.cpp
clang/test/Driver/fuse-ld.c
MaskRay added inline comments.
Comment at: clang/test/Driver/fuse-ld.c:7
+// RUN: cd "%S" && %clang %s -### \
+// RUN: -fuse-ld=Inputs/fuse_ld_windows/ld.foo.exe 2>&1 \
This changes the working directory. It may be better moving cd tests to the
bottom.
keith created this revision.
keith added reviewers: kastiglione, MaskRay, theraven, rnk.
Herald added a project: clang.
This adds support for resolving linker executable paths relatively
instead of requiring them to be names, or absolute paths. This is useful
if you're vendoring an in-tree
