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

[xin liu]

================
@@ -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.
----------------
navyxliu wrote:

Okay. I will drop this change and move to the alternative patch.

My approach here is hacky. It attempts to run a mini-liveness to overwrite the result from liveness analysis. i don't like because it mixes up Transforms and Analysis. 


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