[Mlir-commits] [mlir] [MLIR] Fix use-after-move for DEBUG builds, and broken assert logic. (PR #164763)

[Mehdi Amini]

================
@@ -159,14 +159,19 @@ IterationGraphSorter::IterationGraphSorter(
       loop2OutLvl(loop2OutLvl), iterTypes(std::move(iterTypes)),
       strategy(strategy) {
----------------
joker-eph wrote:

> and has at least some negative effect -- code devs often use the matching variable names at the call sites

What kind of tool and what would be the effect?
Most of my call-sites are not using the same variable names in general, I'm not sure why a tool should complain about this.

(also this ctor is private to the class)

https://github.com/llvm/llvm-project/pull/164763