[PATCH] D146199: [LoopVectorize] Don't tail-fold for scalable VFs when there is no scalar tail

Thu Mar 16 08:30:39 PDT 2023

reames accepted this revision.
reames added a comment.
This revision is now accepted and ready to land.

LGTM

I do wonder if we should consider redefining vscale in LangRef to be a power of two.  I don't think we have a motivating example for a non power of two.  I believe this was originally to support SVE whose original version didn't mandate power of two vector sizes, but I believe a later version of that specification mandated them didn't it?

================
Comment at: llvm/test/Transforms/LoopVectorize/AArch64/eliminate-tail-predication.ll:1
 ; RUN: opt -passes=loop-vectorize -force-target-instruction-cost=1 -prefer-predicate-over-epilogue=predicate-dont-vectorize -S < %s 2>&1 | FileCheck %s

----------------
Please use update-test-checks.py

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

https://reviews.llvm.org/D146199