https://github.com/joker-eph approved this pull request. Can we keep the MLIR header and just define the `using llvm::LogicalResult;` there in the meantime? It'll avoid breaking the world downstream for now. https://github.com/llvm/llvm-project/pull/97309