[llvm] [InstCombine] Canonicalize more saturated add variants (PR #100008)

via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 28 11:28:06 PDT 2024


AtariDreams wrote:

> Please replace tests using something like uge C with ugt C-1 so the input is canonical. And then remove all the tests that become duplicates as a result.
> 
> Your alive2 proofs also include some preconditions -- some cases check that C!=0. But it does not look like your implementation contains those checks?

It does now!

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


More information about the llvm-commits mailing list