[PATCH] D21560: Relax the clearance calculating for breaking partial register dependency.
Sanjay Patel via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 22 12:47:16 PDT 2016
spatel added a comment.
Since even the code comments acknowledge that this is a repeated magic number, please give it a name. Even better would be to make it a cl::opt, so we can run experiments more easily.
FWIW, I don't see any measurable perf difference for the example test case (compiled with -O2) running on Haswell.
Is there already a regression test to show that we're not adding instructions if we're in MinSize mode?
http://reviews.llvm.org/D21560
More information about the llvm-commits
mailing list