[PATCH] D74749: ThinLTOBitcodeWriter: drop dso_local when a GlobalVariable is converted to a declaration
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 5 18:35:16 PST 2020
This revision was automatically updated to reflect the committed changes.
Closed by commit rG952ee0df9ef9: ThinLTOBitcodeWriter: drop dso_local when a GlobalVariable is converted to a… (authored by MaskRay).
Changed prior to commit:
https://reviews.llvm.org/D74749?vs=248558&id=248641#toc
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74749/new/
https://reviews.llvm.org/D74749
Files:
llvm/lib/Transforms/IPO/FunctionImport.cpp
llvm/test/LTO/Resolution/X86/not-prevailing-variables.ll
llvm/test/Transforms/ThinLTOBitcodeWriter/split-dsolocal.ll
Index: llvm/test/Transforms/ThinLTOBitcodeWriter/split-dsolocal.ll
===================================================================
--- /dev/null
+++ llvm/test/Transforms/ThinLTOBitcodeWriter/split-dsolocal.ll
@@ -0,0 +1,23 @@
+; RUN: opt -thinlto-bc -thinlto-split-lto-unit -o %t %s
+; RUN: llvm-modextract -b -n 0 -o %t0.bc %t
+; RUN: llvm-modextract -b -n 1 -o %t1.bc %t
+; RUN: llvm-dis -o - %t0.bc | FileCheck --check-prefix=M0 %s
+; RUN: llvm-dis -o - %t1.bc | FileCheck --check-prefix=M1 %s
+
+; M0: @default = external constant [1 x i8]
+; M0: @hidden = external hidden constant [1 x i8]
+; M0: @al = external global [1 x i8]
+
+; M1: @default = dso_local constant [1 x i8] c"0", !type !0
+; M1: @hidden = hidden constant [1 x i8] c"0", !type !0
+; M1: @al = dso_local alias [1 x i8], [1 x i8]* @default
+ at default = dso_local constant [1 x i8] c"0", !type !0
+ at hidden = dso_local hidden constant [1 x i8] c"0", !type !0
+
+ at al = dso_local alias [1 x i8], [1 x i8]* @default
+
+define [1 x i8]* @f_default() { ret [1 x i8]* @default }
+define [1 x i8]* @f_hidden() { ret [1 x i8]* @hidden }
+define [1 x i8]* @f_al() { ret [1 x i8]* @al }
+
+!0 = !{i32 0, !"typeid"}
Index: llvm/test/LTO/Resolution/X86/not-prevailing-variables.ll
===================================================================
--- llvm/test/LTO/Resolution/X86/not-prevailing-variables.ll
+++ llvm/test/LTO/Resolution/X86/not-prevailing-variables.ll
@@ -7,12 +7,14 @@
; var2 is not prevailing and here we check it is not inlined.
; Check 'var2' was not inlined.
-; RUN: llvm-objdump -d %t2.o.1 | FileCheck %s
+; RUN: llvm-objdump -d -r %t2.o.1 | FileCheck %s
; CHECK: <testVar1>:
; CHECK-NEXT: movl $10, %eax
; CHECK-NEXT: retq
; CHECK: <testVar2>:
-; CHECK-NEXT: movl (%rip), %eax
+; CHECK-NEXT: movq (%rip), %rax
+; CHECK-NEXT: R_X86_64_GOTPCREL var2-0x4
+; CHECK-NEXT: movl (%rax), %eax
; CHECK-NEXT: retq
; Check 'var2' is undefined.
Index: llvm/lib/Transforms/IPO/FunctionImport.cpp
===================================================================
--- llvm/lib/Transforms/IPO/FunctionImport.cpp
+++ llvm/lib/Transforms/IPO/FunctionImport.cpp
@@ -992,6 +992,8 @@
GV.replaceAllUsesWith(NewGV);
return false;
}
+ if (!GV.isImplicitDSOLocal())
+ GV.setDSOLocal(false);
return true;
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74749.248641.patch
Type: text/x-patch
Size: 2333 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200306/880d73bb/attachment.bin>
More information about the llvm-commits
mailing list