[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

Rainer Orth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 22 05:03:54 PDT 2020


ro added a comment.

This patch introduced a couple of failures on the Solaris buildbots (Solaris/sparcv9 <http://lab.llvm.org:8014/builders/clang-solaris11-sparcv9> and Solaris/x86 <http://lab.llvm.org:8014/builders/clang-solaris11-amd64>).

E.g.

  FAIL: Clang :: Driver/env.c (12768 of 69452)
  [...]
  /vol/llvm/src/llvm-project/dist/clang/test/Driver/env.c:21:21: error: CHECK-LD-32-NOT: excluded string found in input
  // CHECK-LD-32-NOT: warning:
                      ^
  <stdin>:5:8: note: found here
  clang: warning: '-fuse-ld=' taking a path is deprecated. Use '--ld-path=' instead
         ^~~~~~~~

For reasons explained in D84029 <https://reviews.llvm.org/D84029>, one needs to specify the Solaris linker, either via `cmake -DCLANG_DEFAULT_LINKER` (so far) or implicitly
as in that patch.

I strongly expect all other uses of `CLANG_DEFAULT_LINKER` to be broken similarly.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83015/new/

https://reviews.llvm.org/D83015





More information about the cfe-commits mailing list