[Mlir-commits] [mlir] [MLIR][Arith] SelectOp fix invalid folding (PR #117555)

[llvmlistbot at llvm.org]

7FM wrote:

> > > We should add a LIT test for the case that used to fail.
> > 
> > 
> > I agree, but at first glance, I couldn't find an upstream MLIR dialect that supports unsigned/signed constants to trigger this bug. But I might have missed a suitable dialect.
> 
> How about adding a new test op to the test dialect? We already use `test.with_bounds` in arith tests.

Thanks for the hint! I didn't know that such a dialect existed.

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