[all-commits] [llvm/llvm-project] 499014: [WoA][MSVC] Use default linker setting in MSVC-com...

maxim-kuvyrkov via All-commits all-commits at lists.llvm.org
Fri Mar 19 11:47:53 PDT 2021


  Branch: refs/heads/release/12.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 4990141a4366eb00abdc8252d7cbb8adeacb9954
      https://github.com/llvm/llvm-project/commit/4990141a4366eb00abdc8252d7cbb8adeacb9954
  Author: Maxim Kuvyrkov <maxim.kuvyrkov at linaro.org>
  Date:   2021-03-19 (Fri, 19 Mar 2021)

  Changed paths:
    M clang/lib/Driver/ToolChains/MSVC.cpp
    M clang/test/Driver/Xlinker-args.c
    M clang/test/Driver/cl-inputs.c
    M clang/test/Driver/cl-link-at-file.c
    M clang/test/Driver/cl-link.c
    M clang/test/Driver/msvc-link.c
    M clang/test/OpenMP/linking.c

  Log Message:
  -----------
  [WoA][MSVC] Use default linker setting in MSVC-compatible driver [take 2]

At the moment "link.exe" is hard-coded as default linker in MSVC.cpp,
so there's no way to use LLD as default linker for MSVC driver.

This patch adds checking of CLANG_DEFAULT_LINKER to MSVC.cpp and
updates unit-tests that expect link.exe linker to explicitly select it
via -fuse-ld=link, so that buildbots and other builds that set
-DCLANG_DEFAULT_LINKER=foobar don't fail these tests.

This is a squash of
- https://reviews.llvm.org/D98493 (MSVC.cpp change) and
- https://reviews.llvm.org/D98862 (unit-tests change)

Fixes https://bugs.llvm.org/show_bug.cgi?id=49624

Reviewed By: maxim-kuvyrkov

Differential Revision: https://reviews.llvm.org/D98935

(cherry-picked from commit 2049fe58903b68f66872a18e608f40e5233b55fb)




More information about the All-commits mailing list