[all-commits] [llvm/llvm-project] fd9f45: [ThinLTO] Don't convert functions to declarations ...

[Shilei Tian via All-commits]

  Branch: refs/heads/users/shiltian/dont-convert-to-decl-if-force-import-all
  Home:   https://github.com/llvm/llvm-project
  Commit: fd9f457f070e5572449481cdde77b4754576c346
      https://github.com/llvm/llvm-project/commit/fd9f457f070e5572449481cdde77b4754576c346
  Author: Shilei Tian <i at tianshilei.me>
  Date:   2025-04-07 (Mon, 07 Apr 2025)

  Changed paths:
    M llvm/lib/Transforms/IPO/FunctionImport.cpp
    A llvm/test/ThinLTO/AMDGPU/Inputs/in-0.ll
    A llvm/test/ThinLTO/AMDGPU/Inputs/in-1.ll
    A llvm/test/ThinLTO/AMDGPU/force-import-all.ll
    A llvm/test/ThinLTO/AMDGPU/lit.local.cfg

  Log Message:
  -----------
  [ThinLTO] Don't convert functions to declarations if `force-import-all` is enabled

On one hand, we intend to `force-import-all` functions when the option is
enabled. On the other hand, we currently drop definitions of some functions and
convert them to declarations, which contradicts this intent.

With this PR, functions will no longer be converted to declarations when
`force-import-all` is enabled.



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