MLIR: Add ReplaceOpAction into DialectConversion and PatternMatch

[mgehre-amd]

Add action to enable listener for RewriterBase::replaceOp, RewriterBase::replaceAllUsesWith and friends.

[cferry-AMD]

Also notify the listener
  /// about every in-place op modification (for every use that was replaced).

Was this not up-to-date? Or is an action something else than the listener?

[jorickert]

My main question is also whats the difference between an action and a listener

[mgehre-amd]

The RewriterBase supports attaching listeners, and the MLIRContext supports action handlers. In this PR, we use the action handlers on the MLIRContext, which allow us to observe across all passes without modifying each.

[cferry-AMD]

replaceAllUsesWith(ValueRange, ValueRange) should also be modified, shouldn't it?

[mgehre-amd]

replaceAllUsesWith(ValueRange, ValueRange)
this calls the replaceAllUsesWith(Value from, Value to), which calls the action here

void replaceAllUsesWith(ValueRange from, ValueRange to) {
    assert(from.size() == to.size() && "incorrect number of replacements");
    for (auto it : llvm::zip(from, to))
      replaceAllUsesWith(std::get<0>(it), std::get<1>(it));
  }

[cferry-AMD]

Thanks for the claraifications, that looks good.