[PATCH] D31821: Remove redundant copy in recurrences

[Taewook Oh via Phabricator via llvm-commits]

twoh added a comment.

@wmi Not at all! Thanks for your comments.

You're right. I should've considered the case where the operands involved in the recurrence cycle live beyond the use in the recurrence cycle. You're suggestion sounds great, but my concern is that computing live range might be expensive because live interval analysis is performed after peephole optimization, What do you think of limit it to the case of operands involved in the recurrence cycle have no use outside of the cycle?


https://reviews.llvm.org/D31821