mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-11-24 03:33:20 +01:00
d41ca54775
dso_local leads to direct access even if the definition is not within this compilation unit (it is still in the same linkage unit). On ELF, such a relocation (e.g. R_X86_64_PC32) referencing a STB_GLOBAL STV_DEFAULT object can cause a linker error in a -shared link. If the linkage is changed to available_externally, the dso_local flag should be dropped, so that no direct access will be generated. The current behavior is benign, because -fpic does not assume dso_local (clang/lib/CodeGen/CodeGenModule.cpp:shouldAssumeDSOLocal). If we do that for -fno-semantic-interposition (D73865), there will be an R_X86_64_PC32 linker error without this patch. Reviewed By: tejohnson Differential Revision: https://reviews.llvm.org/D74751
22 lines
711 B
LLVM
22 lines
711 B
LLVM
; RUN: opt -module-summary %s -o %t1.bc
|
|
; RUN: opt -module-summary %p/Inputs/index-const-prop-define-g.ll -o %t2.bc
|
|
; RUN: llvm-lto2 run -save-temps %t2.bc -r=%t2.bc,g,pl %t1.bc -r=%t1.bc,main,plx -r=%t1.bc,g, -o %t3
|
|
; RUN: llvm-dis %t3.2.3.import.bc -o - | FileCheck %s
|
|
|
|
; The 'store' instruction in @main should prevent internalization
|
|
; even when there is 'load' instruction before it.
|
|
; CHECK: @g = available_externally global i32 42
|
|
|
|
target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128"
|
|
target triple = "x86_64-unknown-linux-gnu"
|
|
|
|
@g = external global i32
|
|
|
|
define i32 @main() {
|
|
%v = load i32, i32* @g
|
|
%q = add i32 %v, 1
|
|
store i32 %q, i32* @g
|
|
|
|
ret i32 %v
|
|
}
|