[PATCH] D30254: Remove sometimes faulty rewrite of memcpy in instcombine.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 23 11:22:36 PST 2017


efriedma added a comment.

> Is it these tests you want me to run?
>  http://llvm.org/docs/TestingGuide.html#test-suite-quickstart

Yes, that's right.

In terms of the results, you should be seeing a page like http://llvm.org/perf/db_default/v4/nts/109028?compare_to=109007 .  On the sidebar on the left, there are two dates in bold: those are the two runs you're comparing.  The differences between the two runs are listed under the heading "Run-Over-Run Changes Detail".  The important number is the first one; the percentage change between the two runs; if it's negative (green), there's a performance improvement, and if it's positive (red), there's a performance regression.  (If there are no significant changes between the two runs, this section might be empty.)

For this patch, please make sure you're comparing the performance of 32-bit binaries (build with "-m32" in CFLAGS/CXXFLAGS).


https://reviews.llvm.org/D30254





More information about the llvm-commits mailing list