[PATCH] D132308: [LV] Replace fixed-order cost model with a SK_Splice shuffle

[Dave Green via Phabricator via llvm-commits]

dmgreen added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/X86/fixed-order-recurrence.ll:7
+
+define void @firstorderrec(ptr nocapture noundef readonly %x, ptr noalias nocapture noundef writeonly %y, i32 noundef %n) {
+; CHECK-LABEL: @firstorderrec(
----------------
RKSimon wrote:
> Is there any change with/without this patch (and/or D132374)?
For firstorderrec there is no difference.
For thirdorderrec below:
 - With D132374 there is no change.
 - Without D132374 the interleaving factor is lower, as the cost of the loop is higher. The VF is still the same.
This patch currently assumes that D132374 goes in first.


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

https://reviews.llvm.org/D132308