김민성 via llvm-dev
2015-Dec-11 06:31 UTC
[llvm-dev] What is the percentage of performance increase for test-suite, which can be considered worth patching?
An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151211/9b33fcd1/attachment.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: 201512111531484_AE6RHZO3.gif Type: image/gif Size: 13168 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151211/9b33fcd1/attachment.gif>
James Molloy via llvm-dev
2015-Dec-11 08:47 UTC
[llvm-dev] What is the percentage of performance increase for test-suite, which can be considered worth patching?
Hi Min, Your question assumes that an acceptance criterion for a patch is that the LLVM test-suite improves. That is incorrect; patches do not need to improve the test-suite in order to be accepted. As a community we care about a very diverse range of workloads. Not all of those (and probably not most!) are represented in the test-suite. What the test-suite aims to do is cover a range of publicly available benchmarks and programs to attempt to give a more representative slice of the "real world" than some other benchmark suites do. It is very often the case that workloads (be they real world applications or benchmarks) have a hot, dominating section of code that if it is optimized well, or in a specific way, the workload improves drastically. It is not expected that all these possible optimizations would improve tests in the test-suite. If they do - fantastic, but that's not a blocker. However, the test-suite can give an indication that a patch is *not* good in the general case. It has a more diverse set of workloads than any other suite I've come across, and so can provide a decent indication of how a patch would behave on "more real world" code than whatever workload you're targetting. It can tell you if your patch even triggers in real world code, and can tell you if it regresses. So regressing the test-suite may block your patch, but improving the test-suite is not a requirement. Cheers, James On Fri, 11 Dec 2015 at 06:32 김민성 via llvm-dev <llvm-dev at lists.llvm.org> wrote:> Hi~ Guys, > > > > Could anyone tell me what is the positive percentage of performance > increase from llvm test-suite, which can be considered worth patching ? > > > > Cheers > > --Min > > > > ------------------------------------------------------ > > *MinSeong KIM* > > Software Engineer > > S/W Platform Development Team > > S.LSI Business > > > > SAMSUNG ELECTRONICS > > Mobile: +82-10-5636-7974 > > E-mail: min.s.kim at samsung.com > > ------------------------------------------------------ > > > > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151211/70a58c5d/attachment-0001.html> -------------- next part -------------- A non-text attachment was scrubbed... Name: 201512111531484_AE6RHZO3.gif Type: image/gif Size: 13168 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151211/70a58c5d/attachment-0002.gif> -------------- next part -------------- A non-text attachment was scrubbed... Name: 201512111531484_AE6RHZO3.gif Type: image/gif Size: 13168 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20151211/70a58c5d/attachment-0003.gif>