[PATCH] D10782: [IndVars] Try to use existing values in RewriteLoopExitValues.

[hfinkel at anl.gov]

hfinkel added a subscriber: hfinkel.
hfinkel accepted this revision.
hfinkel added a reviewer: hfinkel.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: lib/Transforms/Scalar/IndVarSimplify.cpp:529
@@ +528,3 @@
+    if (!match(BB->getTerminator(),
+               m_Br(m_ICmp(Pred, m_Instruction(LHS), m_Instruction(RHS)),
+                    TrueBB, FalseBB)))
----------------
The somewhat arbitrary nature of this heuristic is bothersome, although I can certainly understand why this is a common case.

Nevertheless, I can't think of anything better at the moment.



http://reviews.llvm.org/D10782