================ @@ -30,6 +30,9 @@ namespace looputils { struct InductionVariableInfo { /// the operation allocating memory for iteration variable, mlir::Operation *iterVarMemDef; + /// the operation(s) updating the iteration variable with the current + /// iteration number. + llvm::SetVector<mlir::Operation *> indVarUpdateOps; ---------------- skatrak wrote:
Is there any reason why this has to be a set? It seems like an implementation detail that leaked out of `extractIndVarUpdateOps`. https://github.com/llvm/llvm-project/pull/127634 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits