[PATCH] Enable inlining of recursive functions.

James Molloy james.molloy at arm.com
Tue Jan 27 04:15:21 PST 2015


================
Comment at: lib/Transforms/IPO/Inliner.cpp:64
@@ -63,1 +63,3 @@
 
+// This threshold controls the maximum amount of recursion inlining. Default = 0
+// For direct recursion, this controls the inline levels: if
----------------
This comment does give the information needed, but is fairly difficult to parse. Could you please make it better formatted and elaborate?

For example, I didn't understand what "A1", "A2" etc were until my third reading.

http://reviews.llvm.org/D6996

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list