[all-commits] [llvm/llvm-project] 84ed3c: Revert "[AMDGPU][LTO] Assume closed world after li...

Shilei Tian via All-commits all-commits at lists.llvm.org
Sun Sep 1 06:32:30 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 84ed3c29e8583bdd704d0e2f7e7c3d1162c6181c
      https://github.com/llvm/llvm-project/commit/84ed3c29e8583bdd704d0e2f7e7c3d1162c6181c
  Author: Shilei Tian <i at tianshilei.me>
  Date:   2024-09-01 (Sun, 01 Sep 2024)

  Changed paths:
    M llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp
    M llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
    R llvm/test/LTO/AMDGPU/gpu-rdc-amdgpu-attrs.ll

  Log Message:
  -----------
  Revert "[AMDGPU][LTO] Assume closed world after linking (#105845)" (#106889)

We can't assume closed world even in full LTO post-link stage. It is
only true
if we are building a "GPU executable". However, AMDGPU does support
"dyamic
library". I'm not aware of any approach to tell if it is relocatable
link when
we create the pass. For now let's revert the patch as it is currently
breaking things.
We can re-enable it once we can handle it correctly.



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