[Mlir-commits] [mlir] [MLIR][LivenessAnalysis] Treat a public function as an external (PR #160648)

[Mehdi Amini]

joker-eph wrote:

> The problem is whether it is always possible to find cheaper replacement? What about an aggregated type?

It may not always be possible, but it's always safe to do nothing. 

> if we fail, we need to propagate live arguments in RemoveDeadValues just like PR https://github.com/llvm/llvm-project/pull/160242

Right seems like it may be better handled there: that is "not-live" != "removable" here.

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