[llvm] [SLP] The order of store chains needs to consider the size of the values. (PR #101810)

Alexey Bataev via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 6 03:09:01 PDT 2024


================
@@ -19123,6 +19123,14 @@ bool SLPVectorizerPass::vectorizeStoreChains(BoUpSLP &R) {
     if (V->getPointerOperandType()->getTypeID() >
         V2->getPointerOperandType()->getTypeID())
       return false;
+    if (V->getValueOperand()->getType()->getScalarSizeInBits() <
----------------
alexey-bataev wrote:

No, getTypeID() above compared for pointer operands, which worked for typed pointers. You need to use getTypeID for value operand types

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


More information about the llvm-commits mailing list