[all-commits] [llvm/llvm-project] 76605f: [LinkerWrapper] Make `-Xoffload-linker` match `-Xl...

Joseph Huber via All-commits all-commits at lists.llvm.org
Mon Jul 29 16:11:30 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 76605f5718e07bc0e551620a39a0853cb6fc81f5
      https://github.com/llvm/llvm-project/commit/76605f5718e07bc0e551620a39a0853cb6fc81f5
  Author: Joseph Huber <huberjn at outlook.com>
  Date:   2024-07-29 (Mon, 29 Jul 2024)

  Changed paths:
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/test/Driver/linker-wrapper.c
    M clang/test/Driver/openmp-offload-gpu.c
    M clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp
    M clang/tools/clang-linker-wrapper/LinkerWrapperOpts.td

  Log Message:
  -----------
  [LinkerWrapper] Make `-Xoffload-linker` match `-Xlinker` semantics (#101032)

Summary:
`-Xlinker` is supposed to pass options to the linker, while
`-Xoffload-linker` instead passes it to the `clang` job. This is
unintuitive and results in unnecessarily complex command lines. Because
passing it to the clang job is still useful, I've added
`-device-compiler`. This changes the behavior of the flag, but I think
it should be fine in this case since it's likely rarely used and has a
direct replacement.

---------

Co-authored-by: Joel E. Denny <jdenny.ornl at gmail.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list