[PATCH] D20482: [LoopUnroll] Enable advanced unrolling analysis by default.

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Fri May 20 13:38:10 PDT 2016


mzolotukhin added a comment.

Hi Chandler,

Thanks for LGTM, I'll commit the patch on Monday as I won't be able to check bots on this weekend.

As for the use of profiling info - it indeed looks like the best way to handle this. Also, I think it would make sense to use it more extensively everywhere, not only in loop-unrolling. This way we can save some compile time by not doing expensive optimizations on code that doesn't matter, and then spend more time optimizing relevant pieces of the code. For now it's just general ideas, but I hope to get to something more real soon.

Thanks,
Michael


http://reviews.llvm.org/D20482





More information about the llvm-commits mailing list