[llvm] [InstCombine] Canonicalise SextADD + GEP (PR #69581)

via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 21 20:48:51 PDT 2023


LiqinWeng wrote:

> Please, can you take a bit more care? Your proof is very obviously broken if you just look at the result.
> 
> Here is a fixed version: https://alive2.llvm.org/ce/z/FSNnoo Note that the `nsw` on the `add` is required for correctness, which you don't check.

Pls see: https://alive2.llvm.org/ce/z/vkCK92. The transformation only occurs when the second operand of the add is a constant.
I will add the `nsw` on the `add` 

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


More information about the llvm-commits mailing list