diff options
| author | David Majnemer <david.majnemer@gmail.com> | 2016-04-26 21:04:47 +0000 |
|---|---|---|
| committer | David Majnemer <david.majnemer@gmail.com> | 2016-04-26 21:04:47 +0000 |
| commit | abb9f55c8021c158e5fb9ed95c573648b254e610 (patch) | |
| tree | 4581e5601c74fe3d8da238734e1ce111d6cb0920 /llvm/test | |
| parent | ce26eac51100c294a294ec23a26e7c1c74c9f143 (diff) | |
| download | bcm5719-llvm-abb9f55c8021c158e5fb9ed95c573648b254e610.tar.gz bcm5719-llvm-abb9f55c8021c158e5fb9ed95c573648b254e610.zip | |
Revert "[SimplifyLibCalls] sprintf doesn't copy null bytes"
The destination buffer that sprintf uses is restrict qualified, we do
not need to worry about derived pointers referenced via format
specifiers.
This reverts commit r267580.
llvm-svn: 267605
Diffstat (limited to 'llvm/test')
| -rw-r--r-- | llvm/test/Transforms/InstCombine/sprintf-1.ll | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/llvm/test/Transforms/InstCombine/sprintf-1.ll b/llvm/test/Transforms/InstCombine/sprintf-1.ll index 0c7fbcb2f8e..ddf2f2f88e7 100644 --- a/llvm/test/Transforms/InstCombine/sprintf-1.ll +++ b/llvm/test/Transforms/InstCombine/sprintf-1.ll @@ -65,9 +65,8 @@ define void @test_simplify5(i8* %dst, i8* %str) { %fmt = getelementptr [3 x i8], [3 x i8]* @percent_s, i32 0, i32 0 call i32 (i8*, i8*, ...) @sprintf(i8* %dst, i8* %fmt, i8* %str) ; CHECK-NEXT: [[STRLEN:%[a-z0-9]+]] = call i32 @strlen(i8* %str) -; CHECK-NEXT: call void @llvm.memcpy.p0i8.p0i8.i32(i8* %dst, i8* %str, i32 [[STRLEN]], i32 1, i1 false) -; CHECK-NEXT: [[NUL:%[a-z0-9]+]] = getelementptr i8, i8* %dst, i32 [[STRLEN]] -; CHECK-NEXT: store i8 0, i8* [[NUL]], align 1 +; CHECK-NEXT: [[LENINC:%[a-z0-9]+]] = add i32 [[STRLEN]], 1 +; CHECK-NEXT: call void @llvm.memcpy.p0i8.p0i8.i32(i8* %dst, i8* %str, i32 [[LENINC]], i32 1, i1 false) ret void ; CHECK-NEXT: ret void } |
