[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