[PATCH] D61282: [InlineCost] Check if caller formulates indirect recursion.

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 29 14:53:40 PDT 2019


lebedev.ri added inline comments.


================
Comment at: llvm/test/Transforms/Inline/recursive.ll:70
+; CHECK-INDIRECT: call i32 @large_stack_indirect_recursive_caller
+  %f = call i32 @large_stack_callee(i32 %r)
+; CHECK-INDIRECT: call i32 @large_stack_callee
----------------
Can you also add a test to show what happens if `@large_stack_callee` is not called?
It should then be inlined?
Or is the intention to completely prohibit inlining of recursive calls?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61282/new/

https://reviews.llvm.org/D61282





More information about the llvm-commits mailing list