[PATCH] D130618: [AArch64][LoopVectorize] Enable tail-folding of simple loops on neoverse-v1

[Paul Walker via Phabricator via llvm-commits]

paulwalker-arm accepted this revision.
paulwalker-arm added a comment.
This revision is now accepted and ready to land.

I'm happy with this patch.  It's perhaps not 100% clear cut but on average we're seeing more (important?) wins than losses and there's nothing binding here.  We're in the middle of the LLVM 17 development cycle and thus if evidence presents a more valuable configuration then fine let's use that.  Making the change also gives us more eyes on the testing and benchmarking of tail folding, which I see as a good thing.



================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:42
 
+static cl::opt<unsigned> SVETailFoldInsnThreshold("sve-tailfold-insn-threshold",
+                                                  cl::init(15), cl::Hidden);
----------------
Please can this be `tail-folding` to match the existing command line option naming.


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

https://reviews.llvm.org/D130618