[Mlir-commits] [mlir] [mlir] [dataflow] Refactoring the definition of program points in dat… (PR #105656)

[llvmlistbot at llvm.org]

MaheshRavishankar wrote:

This change seems to have broken `torch-mlir`. I fully expect that this is probably an issue in torch-mlir itself, but The way the data flow framework is used in [`InlineGlobalSlots.cpp`](https://github.com/llvm/torch-mlir/blob/main/lib/Dialect/Torch/Transforms/InlineGlobalSlots.cpp) seems incompatible with this change. I havent been able to make changes to this to get this to build. 

I dont have a repro or anything, and I am not fully upto speed on the DataFlow analysis or how it is used in Torch-MLIR but just leaving a note here that this seems to cause a breakage (due to potentially mis-use of the data flow framework)

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