mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-12-27 22:15:18 +00:00
a106725fc5
patch brings numerous advantages to LLVM. One way to look at it is through diffstat: 109 files changed, 3005 insertions(+), 5906 deletions(-) Removing almost 3K lines of code is a good thing. Other advantages include: 1. Value::getType() is a simple load that can be CSE'd, not a mutating union-find operation. 2. Types a uniqued and never move once created, defining away PATypeHolder. 3. Structs can be "named" now, and their name is part of the identity that uniques them. This means that the compiler doesn't merge them structurally which makes the IR much less confusing. 4. Now that there is no way to get a cycle in a type graph without a named struct type, "upreferences" go away. 5. Type refinement is completely gone, which should make LTO much MUCH faster in some common cases with C++ code. 6. Types are now generally immutable, so we can use "Type *" instead "const Type *" everywhere. Downsides of this patch are that it removes some functions from the C API, so people using those will have to upgrade to (not yet added) new API. "LLVM 3.0" is the right time to do this. There are still some cleanups pending after this, this patch is large enough as-is. llvm-svn: 134829
31 lines
986 B
LLVM
31 lines
986 B
LLVM
; RUN: opt < %s -deadargelim -S > %t
|
|
; RUN: grep {define internal zeroext i32 @test1() nounwind} %t
|
|
; RUN: grep {define internal <{ i32, i32 }> @test2} %t
|
|
|
|
%Ty = type <{ i32, i32 }>
|
|
|
|
; Check if the pass doesn't modify anything that doesn't need changing. We feed
|
|
; an unused argument to each function to lure it into changing _something_ about
|
|
; the function and then changing too much.
|
|
|
|
; This checks if the return value attributes are not removed
|
|
define internal zeroext i32 @test1(i32 %DEADARG1) nounwind {
|
|
ret i32 1
|
|
}
|
|
|
|
; This checks if the struct doesn't get non-packed
|
|
define internal <{ i32, i32 }> @test2(i32 %DEADARG1) {
|
|
ret <{ i32, i32 }> <{ i32 1, i32 2 }>
|
|
}
|
|
|
|
; We use this external function to make sure the return values don't become dead
|
|
declare void @user(i32, <{ i32, i32 }>)
|
|
|
|
define void @caller() {
|
|
%B = call i32 @test1(i32 1)
|
|
%C = call <{ i32, i32 }> @test2(i32 2)
|
|
call void @user(i32 %B, <{ i32, i32 }> %C)
|
|
ret void
|
|
}
|
|
|