[PATCH] D21449: Target independent codesize heuristics for Loop Idiom Recognition

Chad Rosier via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 19 14:49:27 PDT 2016


mcrosier added a comment.

Also, do you have a specific workload that benefits from this series of LIR patches?

I'm not trying to discourage this work, but unless you're targeting something in particular I wonder if this pass is really a good starting point for code size reduction.  My concern is that we'll add a lot of complexity to this pass for little benefit (I believe I made a similar comment in the original patch, but I just wanted to reiterate this point).

Perhaps James has other suggestions.


https://reviews.llvm.org/D21449





More information about the llvm-commits mailing list