[PATCH] D47407: [LoopInstSimplify] Re-implement the core logic of loop-instsimplify to be both simpler and substantially more efficient.

[Michael Zolotukhin via Phabricator via llvm-commits]

mzolotukhin added inline comments.


================
Comment at: llvm/trunk/lib/Transforms/Scalar/LoopInstSimplify.cpp:82
+    for (BasicBlock *BB : RPOT) {
+      for (Instruction &I : *BB) {
+        if (auto *PI = dyn_cast<PHINode>(&I))
----------------
I wonder if it would be more efficient to iterate through `ToSimplify` instead of all instructions in all blocks. What do you think?


Repository:
  rL LLVM

https://reviews.llvm.org/D47407