[PATCH] D101836: [LoopVectorize] Enable strict reductions when allowReordering() returns false

[David Sherwood via Phabricator via llvm-commits]

david-arm added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/scalable-strict-fadd.ll:16
+
+; CHECK-SCALAR: vector.body
 entry:
----------------
Thanks a lot for adding the VF=vscale x 1 case here, but perhaps `CHECK-SCALAR` should be `CHECK-VF1U1`, since we're still vectorising? Also, it's probably worth adding an extra CHECK line for at least one instruction that shows the "<vscale x 1 ..." - maybe the `call float ...` instruction?


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

https://reviews.llvm.org/D101836