================
@@ -975,7 +811,27 @@ void RemoveDeadValues::runOnOperation() {
     }
   });
 
-  cleanUpDeadVals(finalCleanupList);
+  MLIRContext *context = module->getContext();
+  cleanUpDeadVals(context, finalCleanupList);
----------------
rengolin wrote:

> One way to think about this: in case of `RegionBranchOpInterface` ops, we 
> don't erase any values anymore. Instead, we just augment the IR with liveness 
> information (by swapping operands with `ub.poison`). This simplifies the IR 
> good enough for the canonicalization patterns to (incrementally) perform the 
> cleanup.

Ack. Clearer. Thanks!

https://github.com/llvm/llvm-project/pull/173505
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to