[llvm] [SLP]Improve minbitwidth analysis for operands of IToFP and ICmp instructions. (PR #85966)

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 22 07:10:29 PDT 2024


alexey-bataev wrote:

> Hi @alexey-bataev
> 
> I've bisected a miscompile back to this patch. Reproduce with
> 
> ```
> opt -passes=slp-vectorizer bbi-94784.ll -S -o - -mtriple=aarch64 -slp-threshold=-10 -slp-vectorize-hor=0
> ```
> 
> [bbi-94784.ll.gz](https://github.com/llvm/llvm-project/files/15064102/bbi-94784.ll.gz)
> 
> For input %in1=0x3 and %in2=ffff the input function returns 0, but after slp-vectorizer with this patch is returns 2. I think the problem is that after vectorization the "mul" operates on i16 values but it should really be on i64 otherwise the compares with 196605 will go wrong.

Thanks for the reproducer, will double check

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


More information about the llvm-commits mailing list