mirror of
https://github.com/RPCSX/llvm.git
synced 2024-12-12 06:06:19 +00:00
7d2e3aa58e
Summary: We were doing an optimization in the ThinLTO backends of importing constant unnamed_addr globals unconditionally as a local copy (regardless of whether the thin link decided to import them). This should be done in the thin link instead, so that resulting exported references are marked and promoted appropriately, but will need a summary enhancement to mark these variables as constant unnamed_addr. The function import logic during the thin link was trying to handle this proactively, by conservatively marking all values referenced in the initializer lists of exported global variables as also exported. However, this only handled values referenced directly from the initializer list of an exported global variable. If the value is itself a constant unnamed_addr variable, we could end up exporting its references as well. This caused multiple issues. The first is that the transitively exported references weren't promoted. Secondly, some could not be promoted/renamed (e.g. they had a section or other constraint). recursively, instead of just adding the first level of initializer list references to the ExportList directly. Remove this optimization and the associated handling in the function import backend. SPEC measurements indicate we weren't getting much from it in any case. Fixes PR31052. Reviewers: mehdi_amini Subscribers: krasin, llvm-commits Differential Revision: https://reviews.llvm.org/D26880 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@288446 91177308-0d34-0410-b5e6-96231b3b80d8
198 lines
8.1 KiB
LLVM
198 lines
8.1 KiB
LLVM
; First ensure that the ThinLTO handling in llvm-link and llvm-lto handles
|
|
; bitcode without summary sections gracefully.
|
|
; RUN: opt %s -o %t.bc
|
|
; RUN: opt %p/Inputs/funcimport.ll -o %t2.bc
|
|
; RUN: llvm-link %t.bc -summary-index=%t.bc -S
|
|
; RUN: llvm-lto -thinlto -o %t3 %t.bc %t2.bc
|
|
|
|
; Do setup work for all below tests: generate bitcode and combined index
|
|
; RUN: opt -module-summary %s -o %t.bc
|
|
; RUN: opt -module-summary %p/Inputs/funcimport.ll -o %t2.bc
|
|
; RUN: llvm-lto -thinlto -o %t3 %t.bc %t2.bc
|
|
|
|
; Ensure statics are promoted/renamed correctly from this file (all but
|
|
; constant variable need promotion).
|
|
; RUN: llvm-link %t.bc -summary-index=%t3.thinlto.bc -S | FileCheck %s --check-prefix=EXPORTSTATIC
|
|
; EXPORTSTATIC-DAG: @staticvar.llvm.{{.*}} = hidden global
|
|
; Eventually @staticconstvar can be exported as a copy and not promoted
|
|
; EXPORTSTATIC-DAG: @staticconstvar.llvm.0 = hidden unnamed_addr constant
|
|
; EXPORTSTATIC-DAG: @P.llvm.{{.*}} = hidden global void ()* null
|
|
; EXPORTSTATIC-DAG: define hidden i32 @staticfunc.llvm.
|
|
; EXPORTSTATIC-DAG: define hidden void @staticfunc2.llvm.
|
|
|
|
; Ensure that both weak alias to an imported function and strong alias to a
|
|
; non-imported function are correctly turned into declarations.
|
|
; Also ensures that alias to a linkonce function is turned into a declaration
|
|
; and that the associated linkonce function is not in the output, as it is
|
|
; lazily linked and never referenced/materialized.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=globalfunc1:%t.bc -S | FileCheck %s --check-prefix=IMPORTGLOB1
|
|
; IMPORTGLOB1-DAG: define available_externally void @globalfunc1
|
|
; IMPORTGLOB1-DAG: declare void @weakalias
|
|
; IMPORTGLOB1-DAG: declare void @analias
|
|
; IMPORTGLOB1-NOT: @linkoncealias
|
|
; IMPORTGLOB1-NOT: @linkoncefunc
|
|
; IMPORTGLOB1-NOT: declare void @globalfunc2
|
|
|
|
; Ensure that weak alias to a non-imported function is correctly
|
|
; turned into a declaration, but that strong alias to an imported function
|
|
; is imported as alias.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=globalfunc2:%t.bc -S | FileCheck %s --check-prefix=IMPORTGLOB2
|
|
; IMPORTGLOB2-DAG: declare void @analias
|
|
; IMPORTGLOB2-DAG: define available_externally void @globalfunc2
|
|
; IMPORTGLOB2-DAG: declare void @weakalias
|
|
; IMPORTGLOB2-NOT: declare void @globalfunc1
|
|
|
|
; Ensure that strong alias imported in second pass of importing ends up
|
|
; as an alias.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=globalfunc1:%t.bc -import=globalfunc2:%t.bc -S | FileCheck %s --check-prefix=IMPORTGLOB3
|
|
; IMPORTGLOB3-DAG: declare void @analias
|
|
; IMPORTGLOB3-DAG: define available_externally void @globalfunc1
|
|
; IMPORTGLOB3-DAG: define available_externally void @globalfunc2
|
|
; IMPORTGLOB3-DAG: declare void @weakalias
|
|
|
|
; Ensure that strong alias imported in first pass of importing ends up
|
|
; as an alias, and that seeing the alias definition during a second inlining
|
|
; pass is handled correctly.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=globalfunc2:%t.bc -import=globalfunc1:%t.bc -S | FileCheck %s --check-prefix=IMPORTGLOB4
|
|
; IMPORTGLOB4-DAG: declare void @analias
|
|
; IMPORTGLOB4-DAG: define available_externally void @globalfunc2
|
|
; IMPORTGLOB4-DAG: define available_externally void @globalfunc1
|
|
; IMPORTGLOB4-DAG: declare void @weakalias
|
|
|
|
; An alias to an imported function is imported as alias if the function is not
|
|
; available_externally.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=linkoncefunc:%t.bc -S | FileCheck %s --check-prefix=IMPORTGLOB5
|
|
; IMPORTGLOB5-DAG: linkoncealias = alias void (...), bitcast (void ()* @linkoncefunc to void (...)*)
|
|
; IMPORTGLOB5-DAG: define linkonce_odr void @linkoncefunc()
|
|
|
|
; Ensure that imported static variable and function references are correctly
|
|
; promoted and renamed (including static constant variable).
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=referencestatics:%t.bc -S | FileCheck %s --check-prefix=IMPORTSTATIC
|
|
; IMPORTSTATIC-DAG: @staticvar.llvm.{{.*}} = external hidden global
|
|
; Eventually @staticconstvar can be imported as a copy
|
|
; IMPORTSTATIC-DAG: @staticconstvar.llvm.{{.*}} = external hidden unnamed_addr constant
|
|
; IMPORTSTATIC-DAG: define available_externally i32 @referencestatics
|
|
; IMPORTSTATIC-DAG: %call = call i32 @staticfunc.llvm.
|
|
; IMPORTSTATIC-DAG: %0 = load i32, i32* @staticvar.llvm.
|
|
; IMPORTSTATIC-DAG: declare hidden i32 @staticfunc.llvm.
|
|
|
|
; Ensure that imported global (external) function and variable references
|
|
; are handled correctly (including referenced variable imported as
|
|
; available_externally definition)
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=referenceglobals:%t.bc -S | FileCheck %s --check-prefix=IMPORTGLOBALS
|
|
; IMPORTGLOBALS-DAG: @globalvar = external global
|
|
; IMPORTGLOBALS-DAG: declare void @globalfunc1()
|
|
; IMPORTGLOBALS-DAG: define available_externally i32 @referenceglobals
|
|
|
|
; Ensure that common variable correctly imported as common defition.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=referencecommon:%t.bc -S | FileCheck %s --check-prefix=IMPORTCOMMON
|
|
; IMPORTCOMMON-DAG: @commonvar = external global
|
|
; IMPORTCOMMON-DAG: define available_externally i32 @referencecommon
|
|
|
|
; Ensure that imported static function pointer correctly promoted and renamed.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=callfuncptr:%t.bc -S | FileCheck %s --check-prefix=IMPORTFUNCPTR
|
|
; IMPORTFUNCPTR-DAG: @P.llvm.{{.*}} = external hidden global void ()*
|
|
; IMPORTFUNCPTR-DAG: define available_externally void @callfuncptr
|
|
; IMPORTFUNCPTR-DAG: %0 = load void ()*, void ()** @P.llvm.
|
|
|
|
; Ensure that imported weak function reference/definition handled properly.
|
|
; Imported weak_any definition should be skipped with warning, and imported
|
|
; reference should turned into an external_weak declaration.
|
|
; RUN: llvm-link %t2.bc -summary-index=%t3.thinlto.bc -import=callweakfunc:%t.bc -import=weakfunc:%t.bc -S 2>&1 | FileCheck %s --check-prefix=IMPORTWEAKFUNC
|
|
; IMPORTWEAKFUNC-DAG: Ignoring import request for weak-any function weakfunc
|
|
; IMPORTWEAKFUNC-DAG: declare void @weakfunc
|
|
; IMPORTWEAKFUNC-DAG: define available_externally void @callweakfunc
|
|
; IMPORTWEAKFUNC-NOT: @weakvar = extern_weak global i32, align 4
|
|
|
|
@globalvar = global i32 1, align 4
|
|
@staticvar = internal global i32 1, align 4
|
|
@staticconstvar = internal unnamed_addr constant [2 x i32] [i32 10, i32 20], align 4
|
|
@commonvar = common global i32 0, align 4
|
|
@P = internal global void ()* null, align 8
|
|
|
|
@weakalias = weak alias void (...), bitcast (void ()* @globalfunc1 to void (...)*)
|
|
@analias = alias void (...), bitcast (void ()* @globalfunc2 to void (...)*)
|
|
@linkoncealias = alias void (...), bitcast (void ()* @linkoncefunc to void (...)*)
|
|
|
|
define void @globalfunc1() #0 {
|
|
entry:
|
|
ret void
|
|
}
|
|
|
|
define void @globalfunc2() #0 {
|
|
entry:
|
|
ret void
|
|
}
|
|
|
|
define linkonce_odr void @linkoncefunc() #0 {
|
|
entry:
|
|
ret void
|
|
}
|
|
|
|
define i32 @referencestatics(i32 %i) #0 {
|
|
entry:
|
|
%i.addr = alloca i32, align 4
|
|
store i32 %i, i32* %i.addr, align 4
|
|
%call = call i32 @staticfunc()
|
|
%0 = load i32, i32* @staticvar, align 4
|
|
%add = add nsw i32 %call, %0
|
|
%1 = load i32, i32* %i.addr, align 4
|
|
%idxprom = sext i32 %1 to i64
|
|
%arrayidx = getelementptr inbounds [2 x i32], [2 x i32]* @staticconstvar, i64 0, i64 %idxprom
|
|
%2 = load i32, i32* %arrayidx, align 4
|
|
%add1 = add nsw i32 %add, %2
|
|
ret i32 %add1
|
|
}
|
|
|
|
define i32 @referenceglobals(i32 %i) #0 {
|
|
entry:
|
|
%i.addr = alloca i32, align 4
|
|
store i32 %i, i32* %i.addr, align 4
|
|
call void @globalfunc1()
|
|
%0 = load i32, i32* @globalvar, align 4
|
|
ret i32 %0
|
|
}
|
|
|
|
define i32 @referencecommon(i32 %i) #0 {
|
|
entry:
|
|
%i.addr = alloca i32, align 4
|
|
store i32 %i, i32* %i.addr, align 4
|
|
%0 = load i32, i32* @commonvar, align 4
|
|
ret i32 %0
|
|
}
|
|
|
|
define void @setfuncptr() #0 {
|
|
entry:
|
|
store void ()* @staticfunc2, void ()** @P, align 8
|
|
ret void
|
|
}
|
|
|
|
define void @callfuncptr() #0 {
|
|
entry:
|
|
%0 = load void ()*, void ()** @P, align 8
|
|
call void %0()
|
|
ret void
|
|
}
|
|
|
|
@weakvar = weak global i32 1, align 4
|
|
define weak void @weakfunc() #0 {
|
|
entry:
|
|
ret void
|
|
}
|
|
|
|
define void @callweakfunc() #0 {
|
|
entry:
|
|
call void @weakfunc()
|
|
ret void
|
|
}
|
|
|
|
define internal i32 @staticfunc() #0 {
|
|
entry:
|
|
ret i32 1
|
|
}
|
|
|
|
define internal void @staticfunc2() #0 {
|
|
entry:
|
|
ret void
|
|
}
|