[all-commits] [llvm/llvm-project] 681cb5: [Driver] Fix duplicate -L after D150013

Fangrui Song via All-commits all-commits at lists.llvm.org
Fri Jul 7 23:57:58 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 681cb54a54bb60dea9034b4b0b45ccc392875b9a
      https://github.com/llvm/llvm-project/commit/681cb54a54bb60dea9034b4b0b45ccc392875b9a
  Author: Fangrui Song <i at maskray.me>
  Date:   2023-07-07 (Fri, 07 Jul 2023)

  Changed paths:
    M clang/lib/Driver/ToolChains/AMDGPU.cpp
    M clang/lib/Driver/ToolChains/CommonArgs.cpp

  Log Message:
  -----------
  [Driver] Fix duplicate -L after D150013

D150013 is to render -L for AMDGPU but updating tools::AddLinkerInputs is wrong
and causes many non-isCrossCompiling targets to have duplicate -L options
because they do `Args.AddAllArgs(CmdArgs, options::OPT_L);`.

Revert the change and add a `Args.AddAllArgs(CmdArgs, options::OPT_L);` instead.




More information about the All-commits mailing list