mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-12-15 04:30:12 +00:00
fa9d1372c9
r166198 migrated the strcpy optimization to instcombine. The strcpy simplifier that was migrated from Transforms/Scalar/SimplifyLibCalls.cpp was also doing some __strcpy_chk simplifications. Those fortified simplifications were migrated as well, but introduced a bug in the __stpcpy_chk simplifier in the process. This happened because the __strcpy_chk and __stpcpy_chk simplifiers were both mapped to StrCpyChkOpt which was updated with simplifications that worked for __strcpy_chk, but not __stpcpy_chk. This patch fixes the problem by adding proper test coverage and creating a new simplifier for __stpcpy_chk (instead of sharing one with __strcpy_chk). git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@167082 91177308-0d34-0410-b5e6-96231b3b80d8
22 lines
731 B
LLVM
22 lines
731 B
LLVM
; Test that lib call simplification doesn't simplify __stpcpy_chk calls
|
|
; with the wrong prototype.
|
|
;
|
|
; RUN: opt < %s -instcombine -S | FileCheck %s
|
|
|
|
target datalayout = "e-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:32:64-f32:32:32-f64:32:64-v64:64:64-v128:128:128-a0:0:64-f80:128:128"
|
|
|
|
@a = common global [60 x i16] zeroinitializer, align 1
|
|
@.str = private constant [8 x i8] c"abcdefg\00"
|
|
|
|
define void @test_no_simplify() {
|
|
; CHECK: @test_no_simplify
|
|
%dst = getelementptr inbounds [60 x i16]* @a, i32 0, i32 0
|
|
%src = getelementptr inbounds [8 x i8]* @.str, i32 0, i32 0
|
|
|
|
; CHECK-NEXT: call i16* @__strcpy_chk
|
|
call i16* @__strcpy_chk(i16* %dst, i8* %src, i32 8)
|
|
ret void
|
|
}
|
|
|
|
declare i16* @__strcpy_chk(i16*, i8*, i32)
|