[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:

> but it can be confusing for humans(readability and maintainability).

Again: I expect the huge majority of call-sites to not just call a function with matching variable names, can you show an example about how it would become confusing or error-prone?



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