[PATCH] D75318: [mlir] Remove successor operands from the Operation class

[Jacques Pienaar via Phabricator via llvm-commits]

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

Looks good thanks!



================
Comment at: mlir/lib/Transforms/Utils/RegionUtils.cpp:286
+
+    for (unsigned argI = 0, argE = succOperands->size(); argI < argE; argI++) {
       // Iterating args in reverse is needed for correctness, to avoid
----------------
Super nit: ++argI


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75318