[PATCH] D30446: [IndVars] Do not branch on poison
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 15 17:28:16 PDT 2017
efriedma added a comment.
The part that's actually making this code complicated is the attempt to reuse an existing induction variable. Hopefully we don't have to worry about the trip count itself being poison?
Since LSR makes new induction variables anyway, I think the problem this patch is trying to solve doesn't exist if we transform the loop test in LSR. And it would be a bit more powerful there, because we could transform the loop test whether or not the loop actually has a canonical induction variable.
https://reviews.llvm.org/D30446
More information about the llvm-commits
mailing list