[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