[PATCH] D49899: Force test/Driver/fuchsia.c(pp) to use lld

2018-08-06 Thread David Greene via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL339036: Force test/Driver/fuchsia.c(pp) to use lld (authored by greened, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D49899?vs=157689&id=15

[PATCH] D49899: Force test/Driver/fuchsia.c(pp) to use lld

2018-07-31 Thread Petr Hosek via Phabricator via cfe-commits
phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D49899 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D49899: Force test/Driver/fuchsia.c(pp) to use lld

2018-07-27 Thread David Greene via Phabricator via cfe-commits
greened added a comment. In https://reviews.llvm.org/D49899#1178545, @ruiu wrote: > > The Fuchsia driver relies on lld so invoke clang with -fuse-ld=lld. This > > gets the test passing when the clang default linker is something other than > > lld. > > Does it work if lld is not installed at all

[PATCH] D49899: Force test/Driver/fuchsia.c(pp) to use lld

2018-07-27 Thread Rui Ueyama via Phabricator via cfe-commits
ruiu added a comment. > The Fuchsia driver relies on lld so invoke clang with -fuse-ld=lld. This gets > the test passing when the clang default linker is something other than lld. Does it work if lld is not installed at all? I believe if the driver cannot find a specified linker, it reports an