[PATCH] D109309: [LoopFlatten] Make the analysis more robust after IV widening

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopFlatten.cpp:443
+    // Look through extends if the IV has been widened.
+    if (FI.Widened && isa<SExtInst>(MatchedItCount))
+      MatchedItCount = dyn_cast<SExtInst>(MatchedItCount)->getOperand(0);
----------------
Do we know this is always valid? For any sext? Or do we need to check the types match the new IV?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109309/new/

https://reviews.llvm.org/D109309