[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:8
+
+define dso_local i32 @uniform_load(i64 %n, i64 %boff, i64 %coff, ptr nocapture readonly %b, ptr nocapture readnone %c) #0 {
+; CHECK: LAA: Found a loop in uniform_load: for.body
----------------
sdesmalen wrote:
> nit: can you clean up this test a bit? `dso_local` `nocapture`, etc aren't really required for the LV test.
> Also, can you use some constant FP value instead of loading from `@globval` ?
I can try using other types of uniform load, so long as the load is treated as scalar with predication for a fixed-width VF.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128272