Skip to content

Commit d255359

Browse files
authored
[NFC][AMDGPU] Remove amdgpu-link-time-lds module flag (#193806)
We could just use `AMDGPUTargetMachine::EnableObjectLinking` to control. Don't really need this, which is from a previous design where we wanted to let different components to decide whether we want to use object linking independently, but that would not really work.
1 parent df1c7eb commit d255359

5 files changed

Lines changed: 0 additions & 13 deletions

llvm/lib/Target/AMDGPU/AMDGPULowerModuleLDSPass.cpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1057,8 +1057,6 @@ class AMDGPULowerModuleLDS {
10571057
}
10581058
}
10591059

1060-
M.addModuleFlag(Module::Error, "amdgpu-link-time-lds", 1);
1061-
10621060
DenseSet<GlobalVariable *> AllLDSVarsForCleanup = AllReplacedVars;
10631061
AllLDSVarsForCleanup.insert(GlobalScopeVars.begin(), GlobalScopeVars.end());
10641062
removeLocalVarsFromUsedLists(M, AllLDSVarsForCleanup);

llvm/test/CodeGen/AMDGPU/lower-module-lds-link-time-classify.ll

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -51,9 +51,6 @@ declare void @extern_func()
5151
; CHECK-DAG: !{ptr @my_kernel, ptr addrspace(3) @lds_global}
5252
; CHECK-DAG: !{ptr @func, ptr addrspace(3) @lds_global}
5353

54-
; Module should be marked with the link-time LDS module flag.
55-
; CHECK: !{i32 1, !"amdgpu-link-time-lds", i32 1}
56-
5754
define void @func() {
5855
%gep1 = getelementptr [64 x i32], ptr addrspace(3) @lds_global, i32 0, i32 0
5956
store i32 1, ptr addrspace(3) %gep1

llvm/test/CodeGen/AMDGPU/lower-module-lds-link-time-global-scope.ll

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,9 +76,6 @@ declare void @extern_func()
7676
; CHECK-DAG: !{ptr @leaf_func, ptr addrspace(3) @lds_leaf}
7777
; CHECK-DAG: !{ptr @direct_kernel, ptr addrspace(3) @lds_direct}
7878

79-
; Module flag.
80-
; CHECK: !{i32 1, !"amdgpu-link-time-lds", i32 1}
81-
8279
define void @shared_func() {
8380
%gep = getelementptr [64 x i32], ptr addrspace(3) @lds_shared, i32 0, i32 0
8481
store i32 1, ptr addrspace(3) %gep

llvm/test/CodeGen/AMDGPU/lower-module-lds-link-time-internal-func.ll

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,6 @@ declare void @extern_func()
2323
; CHECK: !amdgpu.lds.uses = !{{{![0-9]+}}}
2424
; CHECK-DAG: !{ptr @helper, ptr addrspace(3) @[[STRUCT]]}
2525

26-
; CHECK: !{i32 1, !"amdgpu-link-time-lds", i32 1}
27-
2826
define internal void @helper() {
2927
%gep = getelementptr [32 x i32], ptr addrspace(3) @lds_priv, i32 0, i32 0
3028
store i32 1, ptr addrspace(3) %gep

llvm/test/CodeGen/AMDGPU/lower-module-lds-link-time-internal-multi-user.ll

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,6 @@ source_filename = "source_a.hip"
3030
; CHECK-DAG: !{ptr @kernel1, ptr addrspace(3) @[[INTERN]]}
3131
; CHECK-DAG: !{ptr @kernel2, ptr addrspace(3) @[[INTERN]]}
3232

33-
; Module should be marked with the link-time LDS module flag.
34-
; CHECK: !{i32 1, !"amdgpu-link-time-lds", i32 1}
35-
3633
define amdgpu_kernel void @kernel1() {
3734
%gep_a = getelementptr [32 x i32], ptr addrspace(3) @a, i32 0, i32 0
3835
store i32 1, ptr addrspace(3) %gep_a

0 commit comments

Comments
 (0)