[PATCH] D60565: [LOOPINFO] Extend Loop object to add utilities to get the loop bounds, step, induction variable, and guard branch.

[Whitney via Phabricator via llvm-commits]

Whitney marked an inline comment as done.
Whitney added inline comments.


================
Comment at: llvm/include/llvm/Analysis/LoopInfo.h:579
+    /// Get the instruction which update the loop induction variable.
+    Instruction &getStepInst() const { return StepInst; }
+    /// Get the step that the loop induction variable get updated for each loop
----------------
fhahn wrote:
> There already exists InductionDescriptor with a very similar interface and use case: http://llvm.org/doxygen/classllvm_1_1InductionDescriptor.html#a2dbe0a3e4cb9485251e4741a99e0c33b
> 
> Would it be possible to use it in your case (or integrate it with your LoopBounds struct)?
Thanks for the reminder, I am looking into integrating InductionDescriptor in LoopBounds.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60565