diff options
| author | Tom Stellard <thomas.stellard@amd.com> | 2013-10-24 16:38:33 +0000 |
|---|---|---|
| committer | Tom Stellard <thomas.stellard@amd.com> | 2013-10-24 16:38:33 +0000 |
| commit | bc7d87f07c804555f8b6f7af4d1919de356ae89a (patch) | |
| tree | aa51beada8139902ea84738c47f5631f3b813cef /llvm/test/Transforms | |
| parent | 9f36932c8db1801cb545057f0762fff04d3f0af4 (diff) | |
| download | bcm5719-llvm-bc7d87f07c804555f8b6f7af4d1919de356ae89a.tar.gz bcm5719-llvm-bc7d87f07c804555f8b6f7af4d1919de356ae89a.zip | |
Inliner: Handle readonly attribute per argument when adding memcpy
Patch by: Vincent Lejeune
llvm-svn: 193356
Diffstat (limited to 'llvm/test/Transforms')
| -rw-r--r-- | llvm/test/Transforms/Inline/byval.ll | 33 |
1 files changed, 29 insertions, 4 deletions
diff --git a/llvm/test/Transforms/Inline/byval.ll b/llvm/test/Transforms/Inline/byval.ll index e601faf2bb3..3d41dc5a794 100644 --- a/llvm/test/Transforms/Inline/byval.ll +++ b/llvm/test/Transforms/Inline/byval.ll @@ -25,7 +25,7 @@ entry: store i64 2, i64* %tmp4, align 4 call void @f( %struct.ss* byval %S ) nounwind ret i32 0 -; CHECK: @test1() +; CHECK-LABEL: @test1() ; CHECK: %S1 = alloca %struct.ss ; CHECK: %S = alloca %struct.ss ; CHECK: call void @llvm.memcpy @@ -52,7 +52,7 @@ entry: store i64 2, i64* %tmp4, align 4 %X = call i32 @f2( %struct.ss* byval %S ) nounwind ret i32 %X -; CHECK: @test2() +; CHECK-LABEL: @test2() ; CHECK: %S = alloca %struct.ss ; CHECK-NOT: call void @llvm.memcpy ; CHECK: ret i32 @@ -74,7 +74,7 @@ entry: %S = alloca %struct.ss, align 1 ;; May not be aligned. call void @f3( %struct.ss* byval align 64 %S) nounwind ret void -; CHECK: @test3() +; CHECK-LABEL: @test3() ; CHECK: %S1 = alloca %struct.ss, align 64 ; CHECK: %S = alloca %struct.ss ; CHECK: call void @llvm.memcpy @@ -97,10 +97,35 @@ entry: %S = alloca %struct.ss, align 2 ; <%struct.ss*> [#uses=4] %X = call i32 @f4( %struct.ss* byval align 64 %S ) nounwind ret i32 %X -; CHECK: @test4() +; CHECK-LABEL: @test4() ; CHECK: %S = alloca %struct.ss, align 64 ; CHECK-NOT: call void @llvm.memcpy ; CHECK: call void @g3 ; CHECK: ret i32 4 } +; Inlining a byval struct should NOT cause an explicit copy +; into an alloca if the parameter is readonly + +define internal i32 @f5(%struct.ss* byval readonly %b) nounwind { +entry: + %tmp = getelementptr %struct.ss* %b, i32 0, i32 0 ; <i32*> [#uses=2] + %tmp1 = load i32* %tmp, align 4 ; <i32> [#uses=1] + %tmp2 = add i32 %tmp1, 1 ; <i32> [#uses=1] + ret i32 %tmp2 +} + +define i32 @test5() nounwind { +entry: + %S = alloca %struct.ss ; <%struct.ss*> [#uses=4] + %tmp1 = getelementptr %struct.ss* %S, i32 0, i32 0 ; <i32*> [#uses=1] + store i32 1, i32* %tmp1, align 8 + %tmp4 = getelementptr %struct.ss* %S, i32 0, i32 1 ; <i64*> [#uses=1] + store i64 2, i64* %tmp4, align 4 + %X = call i32 @f5( %struct.ss* byval %S ) nounwind + ret i32 %X +; CHECK-LABEL: @test5() +; CHECK: %S = alloca %struct.ss +; CHECK-NOT: call void @llvm.memcpy +; CHECK: ret i32 +} |
