[PATCH] D13266: SLPVectorizer: limit the scheduling region size per basic block to avoid long compile times for very large blocks.

Erik Eckstein via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 30 10:14:21 PDT 2015


eeckstein added a comment.

Thanks for reviewing!
I committed it in r248917.

> BTW, would it make sense to add a test to check that we properly use MinScheduleRegionSize too?


It's not so important than checking that everything does well when the limit is exceeded. But yes, why not?
I'll add a test in a separate commit.


http://reviews.llvm.org/D13266





More information about the llvm-commits mailing list