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

[Chad Rosier via llvm-commits]

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