[PATCH] D105632: [LV] Use lookThroughAnd with logical reductions

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 16 08:39:56 PDT 2021


david-arm added inline comments.


================
Comment at: llvm/test/Transforms/LoopVectorize/trunc-reductions.ll:150
+; CHECK-LABEL: @reduction_smin_trunc(
+; CHECK-NEXT:  entry:
+; CHECK-NEXT:    br label [[FOR_BODY:%.*]]
----------------
nit: Hi @kmclaughlin, sorry to be a pain, but perhaps it's worth having a single CHECK line for all the min/max tests, i.e.

CHECK-NOT: vector.body
CHECK-NOT: <8 x

instead of checking for the same scalar loop?


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

https://reviews.llvm.org/D105632



More information about the llvm-commits mailing list