[PATCH] D146128: [SVE][LoopVectorize] Add option to disable tail-folding for reverse loops

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 15 09:57:57 PDT 2023


paulwalker-arm added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:544
+                                   InterleavedAccessInfo *IAI,
+                                   bool NeedsReversePred) const;
 
----------------
If the number of factors needed to determine the predication strategy is going to increase, perhaps it's worth creating a descriptor class, much like IntrinsicCostAttributes, to keep the interface churn down.

What do people think?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146128



More information about the llvm-commits mailing list