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

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 3 17:39:15 PDT 2016


Hi James, I have some pending comments with the patch.

David

On Thu, Nov 3, 2016 at 1:39 AM, James Molloy <james.molloy at arm.com> wrote:

> jmolloy added a comment.
>
> 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?
>
> Cheers,
>
> James
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D24338
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161103/9fc96299/attachment.html>


More information about the llvm-commits mailing list