[llvm] r311228 - [Inliner] Clean up a test case a bit to make it more clear what is being
Chandler Carruth via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 18 23:06:44 PDT 2017
Author: chandlerc
Date: Fri Aug 18 23:06:44 2017
New Revision: 311228
URL: http://llvm.org/viewvc/llvm-project?rev=311228&view=rev
Log:
[Inliner] Clean up a test case a bit to make it more clear what is being
tested and why.
Modified:
llvm/trunk/test/Transforms/Inline/recursive.ll
Modified: llvm/trunk/test/Transforms/Inline/recursive.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Transforms/Inline/recursive.ll?rev=311228&r1=311227&r2=311228&view=diff
==============================================================================
--- llvm/trunk/test/Transforms/Inline/recursive.ll (original)
+++ llvm/trunk/test/Transforms/Inline/recursive.ll Fri Aug 18 23:06:44 2017
@@ -1,39 +1,42 @@
+; Inlining in the presence of recursion presents special challenges that we
+; test here.
+;
; RUN: opt -inline -S < %s | FileCheck %s
; RUN: opt -passes='cgscc(inline)' -S < %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"
-target triple = "i386-apple-darwin10.0"
-
-; rdar://10853263
-
-; Make sure that the callee is still here.
-; CHECK-LABEL: define i32 @callee(
-define i32 @callee(i32 %param) {
+define i32 @large_stack_callee(i32 %param) {
+; CHECK-LABEL: define i32 @large_stack_callee(
+entry:
%yyy = alloca [100000 x i8]
%r = bitcast [100000 x i8]* %yyy to i8*
- call void @foo2(i8* %r)
+ call void @bar(i8* %r)
ret i32 4
}
-; CHECK-LABEL: define i32 @caller(
+; Test a recursive function which calls another function with a large stack. In
+; addition to not inlining the recursive call, we should also not inline the
+; large stack allocation into a potentially recursive frame.
+define i32 @large_stack_recursive_caller(i32 %param) {
+; CHECK-LABEL: define i32 @large_stack_recursive_caller(
+entry:
; CHECK-NEXT: entry:
; CHECK-NOT: alloca
-; CHECK: ret
-define i32 @caller(i32 %param) {
-entry:
%t = call i32 @foo(i32 %param)
%cmp = icmp eq i32 %t, -1
br i1 %cmp, label %exit, label %cont
cont:
- %r = call i32 @caller(i32 %t)
- %f = call i32 @callee(i32 %r)
- br label %cont
+ %r = call i32 @large_stack_recursive_caller(i32 %t)
+; CHECK: call i32 @large_stack_recursive_caller
+ %f = call i32 @large_stack_callee(i32 %r)
+; CHECK: call i32 @large_stack_callee
+ br label %exit
+
exit:
ret i32 4
}
-declare void @foo2(i8* %in)
+declare void @bar(i8* %in)
declare i32 @foo(i32 %param)
More information about the llvm-commits
mailing list