[PATCH] Enable inlining of recursive functions.
pablo.barrio at arm.com
Thu Jan 15 09:58:24 PST 2015
James, thank you for the review,
I uploaded the required changes to the patch with some verbose inlined comments. I could not find more succinct explanations, sorry about that. Also, forcing me to find an explanation about the inline levels made me realize that I was inlining one more time than expected (3 times for a maximum level of 2). Code and regression test fixed.
About treating recursion inlining as loop unrolling, it makes sense. Here I deliberately stayed away from the InlineCost analysis, so the heuristics have not been changed at all. However, it is certainly something that we want to model in the cost analysis.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 10032 bytes
Desc: not available
More information about the llvm-commits