[PATCH] D95245: [SVE] Add support for scalable vectorization of loops with int/fast FP reductions

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h:190-191
+  bool isLegalScalarTypeForSVE(Type *Ty) const {
+    if (Ty->isBFloatTy() && ST->hasBF16())
+      return true;
+
----------------
I forgot to mention that there are no reduction instructions for bfloat, so you'll need to catch out that specific case in `isLegalToVectorizeReduction`


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

https://reviews.llvm.org/D95245