[Mlir-commits] [mlir] [mlir][arith] Gate min/max expansion in arith-expand behind include-min-max (PR #211884)

[Md Abdullah Shahneous Bari]

mshahneo wrote:

> 2\. If there's also a lowering - or a lowering you can easily add - for these min/max ops to SPIR-V, I'd go ahead and change the default behavior.

Not really, I actually kept it not break anything.

>If there's also a lowering - or a lowering you can easily add - for these min/max ops to SPIR-V, I'd go ahead and change the default behavior. I'm willing to sign off on the breaking change here.

I am so sorry, I realized that I didn't respond to this, SPIR-V already has a lowering. And if you'd like I'd change the default to `false`.

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