[PATCH] D137639: [SLP]Fix PR58863: Mask index beyond mask size for non-power-2 insertelement analysis.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 8 06:59:40 PST 2022


RKSimon added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:6600
+          }
           Mask[I] = InMask.test(I) ? UndefMaskElem : I;
+        }
----------------
Is this any better?
```
Mask[I] = ((I >= InMask.size()) || InMask.test(I)) ? UndefMaskElem : I;
```


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137639/new/

https://reviews.llvm.org/D137639



More information about the llvm-commits mailing list