[Mlir-commits] [mlir] [mlir][tosa] Add NaN Propagation Mode Support (PR #121951)

[Luke Hutton]

================
@@ -361,7 +361,7 @@ struct ClampClampOptimization : public OpRewritePattern<tosa::ClampOp> {
           op, op.getType(), clampOp.getInput(),
           rewriter.getI64IntegerAttr(minInt),
           rewriter.getI64IntegerAttr(maxInt), rewriter.getF32FloatAttr(minFp),
-          rewriter.getF32FloatAttr(maxFp));
+          rewriter.getF32FloatAttr(maxFp), rewriter.getStringAttr("IGNORE"));
----------------
lhutton1 wrote:

cc @tatwaichong, does this handling seem reasonable to you?

If we're unsure how this should be handled at this time, perhaps we can avoid performing this optimization for every case other than the default (PROPAGATE, PROPAGATE).

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