[PATCH] D17584: Cleanup inline cost analysis code

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 15:28:24 PDT 2016


eraman added a comment.

Submitted the second part. Instead of the third part, I think it is better to just return when Cost > Threshold instead of breaking out of the loop. Moving the check inside if (SingleBB && TI->getNumSuccessors() > 1) reduces the readability IMO.


http://reviews.llvm.org/D17584





More information about the llvm-commits mailing list