[llvm-commits] [PATCH] Take into account minimize size attribute in the inliner

Quentin Colombet qcolombet at apple.com
Tue Dec 11 21:33:42 PST 2012


On Dec 11, 2012, at 7:51 PM, Eric Christopher <echristo at gmail.com> wrote:

> 
> Thanks for the review, I'll add a testcase.
> 
> Awesome.
Attached a patch for the test case (separate patch as this one will go into clang).
>  
> Regarding the performance impact, tell me what do you want me to measure/run.
> 
> 
>  Since it probably only affects -Oz input a timing/size run of projects/test-suite with Oz before and after should be sufficient to show some size wins and what kind of performance impact we're looking at. You may also want to run SPEC as well from the extras repo.

Thanks, I'll look at that.

-Quentin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121211/dc956816/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: inline-minsize_testcase.patch
Type: application/octet-stream
Size: 1335 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121211/dc956816/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121211/dc956816/attachment-0001.html>


More information about the llvm-commits mailing list