[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