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

> My point was that In terms of risk,

Right, that's why I'm trying to get to the bottom of it, it's mostly an academic exercise, but the underlying point is "we shouldn't be afraid of changes", so understanding the source of what makes you reluctant looks interesting to me :)

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