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

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 4 00:37:41 PDT 2016


chandlerc added a comment.

In https://reviews.llvm.org/D24338#586539, @jmolloy wrote:

> Hi Mehdi,
>
> Thanks! Would you prefer I strip that out here and submit another phab review, or is it OK to split them apart upon committing?


I'd go ahead and strip them out so the review can continue without conflating them? (Mostly because the review seems to continue...)

I'm even fine if you want to land that part of this first, and then clean up the layering with TTI, but should check with David.


Repository:
  rL LLVM

https://reviews.llvm.org/D24338





More information about the llvm-commits mailing list