[Mlir-commits] [mlir] [mlir][arith] Fold ceildivsi with MININT operands (PR #214637)

[Hung Kuan Tseng]

Tim096 wrote:

@kuhar gentle ping -- this has been open since Aug 7 without a review.

Since it was opened, premerge has run and is green: Build and Test on Linux,
Windows and Linux AArch64, plus `code_formatter` and the LLVM_ABI check. It
still merges cleanly against main, and the patch itself has not changed.

The one part that wants a reviewer's eye is the only existing test this
touches: `@simple_arith.ceildivsi_overflow` is renamed to
`ceildivsi_minint_dividend`, because the case it pinned as "not folded" is now
folded. The description calls this out, but it is the single behaviour change
to a pre-existing test -- everything else in the test file is new.

No rush if the queue is long; I would just rather it not fall off the list.


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