[PATCH] D128272: [LoopVectorize] Change PredicatedBBsAfterVectorization to be per VF

[David Sherwood via Phabricator via llvm-commits]

david-arm added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/sve-tail-folding-cost.ll:16
+
+for.body:                                         ; preds = %for.body, %for.body
+  %indvars.iv = phi i64 [ 1, %entry ], [ %indvars.iv.next, %for.body ]
----------------
sdesmalen wrote:
> What in this loop is causing `PredicatedBBsAfterVectorization` to be `true` for fixed-width vectors?
It's the uniform load of `@globval`, which causes LoopVectorizationCostModel::collectInstsToScalarize to insert the BB into PredicatedBBsAfterVectorization.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128272