[PATCH] D134614: [SCEV] Support clearing Block/LoopDispositions for a single value.

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 05:04:27 PDT 2022


nikic accepted this revision.
nikic added a comment.
This revision is now accepted and ready to land.

LG



================
Comment at: llvm/lib/Analysis/ScalarEvolution.cpp:8418
+
+  LoopDispositions.erase(S);
 }
----------------
This line shouldn't be there?


================
Comment at: llvm/lib/Transforms/Scalar/LoopDeletion.cpp:92
 
+      bool InstrMoved = false;
       if (Instruction *I = dyn_cast<Instruction>(incoming)) {
----------------
Declaration can be moved inside the if.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134614



More information about the llvm-commits mailing list