[Mlir-commits] [mlir] [MLIR][RemoveDeadValues] Mark arguments of a public function Live (PR #160242)

[Mehdi Amini]

================
@@ -852,8 +887,13 @@ void RemoveDeadValues::runOnOperation() {
     } else if (isa<CallOpInterface>(op)) {
       // Nothing to do because this op is associated with a function op and gets
       // cleaned when the latter is cleaned.
+      //
+      // The only exception is public callee. By default, Liveness analysis is inter-procedural.
+      // Unused arguments of a public function nonLive and are propagated to the caller.
+      // processCallOp puts them to liveVals.
----------------
joker-eph wrote:

I am not sure I'm understanding the comment?

MaybeL

```suggestion
      // The only exception is public callee. Arguments of a public function are all live
      // must be propagated to the caller as such.
```

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