[Mlir-commits] [mlir] [mlir][NFC] Move LLVM::ModuleTranslation::SaveStack to a shared header (PR #144897)

[Oleksandr Alex Zinenko]

ftynse wrote:

@tblah this PR had a layering violation that I overlooked: files in lib/Support must not depend on mlir/IR. It was "fixed" by https://github.com/llvm/llvm-project/pull/145598 instead of reverting... I think a cleaner fix would have been to move `WalkResult` to lib/Support because neither it nor the stack logic depends on IR constructs.

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