[Mlir-commits] [mlir] [mlir][arith] Fix overflow bug in arith::CeilDivSIOp::fold (PR #90947)

[Jacob Yu]

pingshiyu wrote:

I'm a little confused here (mostly due to my own ignorance): the original bug was on the conversion pass `--convert-arith-to-llvm`, but this patch is on the constant folder. Just wondering if you could point out where in the codebase is the constant folder called duirng conversion?

Perhaps an extra integration test, consisting of the original file with `arith-expand` or `convert-arith-to-llvm` pass, would give a bit further assurance?

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