[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