[PATCH] D22120: Move inline threshold related flags to InlineSimple.cpp

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 29 20:43:18 PDT 2016


mehdi_amini added a subscriber: mehdi_amini.

================
Comment at: include/llvm/Analysis/InlineCost.h:38
@@ +37,3 @@
+
+// Use when -O[34] is specified.
+const int OptAggressiveThreshold = 275;
----------------
There is no O4, did you mean `O{1,2,3}` here?

================
Comment at: include/llvm/Analysis/InlineCost.h:115
@@ +114,3 @@
+/// commandline
+/// options.
+struct InlineParams {
----------------
You have a funky line return now


https://reviews.llvm.org/D22120





More information about the llvm-commits mailing list