[PATCH] D31821: Remove redundant copy in recurrences

Taewook Oh via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 21 11:39:48 PDT 2017


twoh added a comment.

@MatzeB I was considering other passes as well for the sake of simplicity, but I concluded two-address instruction is the right place to do. The problem doesn't appear while we're in SSA, but becomes an issue when we make an decision about which operand register should be the destination register. I think this is why isProfitableToCommute function is in two-address instruction pass.


https://reviews.llvm.org/D31821





More information about the llvm-commits mailing list