[PATCH] D139934: [IndVars] Apply more optimistic SkipLastIter for AND/OR conditions

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 17 04:15:03 PST 2023


nikic added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/IndVarSimplify.cpp:1490
+      for (size_t j = 0; j < LeafConditions.size(); j++)
+        if (i != j) {
+          auto EL = SE->computeExitLimitFromCond(L, LeafConditions[j], Inverted,
----------------
I don't really get why we need this quadratic-complexity code. It doesn't look like anything inside here depends on the outer `i`?


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

https://reviews.llvm.org/D139934



More information about the llvm-commits mailing list