[PATCH] D124284: [SLP]Try partial store vectorization if supported by target.

[Valeriy Dmitriev via Phabricator via llvm-commits]

vdmitrie added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/BasicTTIImpl.h:330
+    };
+    while (VF > 2 && IsSupportedByTarget(VF))
+      VF /= 2;
----------------
Hm, there is some discrepancy here.
Let's assume we entered the loop with VF = 8  and target supports 8 but not 4.

VF == 8 , VF >2 && IsSupportedByTarget(8) evaluates to true =>  VF = VF/2
VF == 4 , VF >2 && IsSupportedByTarget(4) evaluates to false

hence we return 4 which is actually not supported by target. Is that what the intent was?



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124284