[Mlir-commits] [mlir] [mlir][arith] Fix overflow bug in arith::CeilDivSIOp::fold (PR #90947)
Mehdi Amini
llvmlistbot at llvm.org
Mon May 6 03:02:06 PDT 2024
https://github.com/joker-eph approved this pull request.
This seems to me like it makes the folder strictly more conservative, which should be safe.
It is still open whether we couldn't be more aggressive though: are there cases where we could fold but we don't right now? (if so can you add a TODO? Possibly a test for these?)
Can we accurately catch the real overflow cases and folds to poison for example? (arith may not generate poison out-of-thin air for now though)
https://github.com/llvm/llvm-project/pull/90947
More information about the Mlir-commits
mailing list