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

David Li via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 29 20:47:49 PDT 2016


davidxl added inline comments.

================
Comment at: include/llvm/Analysis/InlineCost.h:38
@@ +37,3 @@
+
+// Use when -O[34] is specified.
+const int OptAggressiveThreshold = 275;
----------------
mehdi_amini wrote:
> There is no O4, did you mean `O{1,2,3}` here?
Clang driver maps O4 to O3 (so O4 is actually 'supported').


https://reviews.llvm.org/D22120





More information about the llvm-commits mailing list