[PATCH] D24338: [InlineCost] Remove CallPenalty and change MinSizeThreshold to 5

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 08:47:37 PDT 2016


mehdi_amini added a comment.

Thanks, that looks good as a first step! See one inline comment.



================
Comment at: lib/Analysis/InlineCost.cpp:1272
+  Cost -= InlineConstants::InstrCost;
+  Cost -= CallPenalty;
+  
----------------
I think this should be a separate patch, all the rest of this revision would be NFC


Repository:
  rL LLVM

https://reviews.llvm.org/D24338





More information about the llvm-commits mailing list