[all-commits] [llvm/llvm-project] e9d517: [Clang] Handle `-flto-partitions` generically and ...

[Joseph Huber via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e9d517d183b18ef6c94e0d0d10d1ac229a4a6014
      https://github.com/llvm/llvm-project/commit/e9d517d183b18ef6c94e0d0d10d1ac229a4a6014
  Author: Joseph Huber <huberjn at outlook.com>
  Date:   2025-03-27 (Thu, 27 Mar 2025)

  Changed paths:
    M clang/include/clang/Driver/Options.td
    M clang/lib/Driver/ToolChains/AMDGPU.cpp
    M clang/lib/Driver/ToolChains/Clang.cpp
    M clang/lib/Driver/ToolChains/CommonArgs.cpp
    M clang/lib/Driver/ToolChains/HIPAMD.cpp
    M clang/test/Driver/amdgpu-toolchain.c
    M clang/test/Driver/hip-toolchain-rdc-flto-partitions.hip
    M clang/test/Driver/hip-toolchain-rdc-static-lib.hip
    M clang/test/Driver/hip-toolchain-rdc.hip

  Log Message:
  -----------
  [Clang] Handle `-flto-partitions` generically and forward it properly (#133283)

Summary:
The https://github.com/llvm/llvm-project/pull/128509 patch introduced
`--flto-partitions`. This was marked as a HIP only argument, and was
also spelled and handled incorrectly for an `-f` option. This patch
makes the handling generic for `ld.lld` consumers.

This also fixes some issues with emitting the flags being put after the
default arguments, preventing users from overriding them. Also, forwards
things properly for the new driver so we can test this.



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