[PATCH] D93030: [AArch64][SVE]Add cost model for masked gather and scatter for scalable vector.

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:355
 
+  Optional<unsigned> getMaxVScale() const { return 1; }
+
----------------
ctetreau wrote:
> Is a concrete 1 here really the best default? I would think that for scalable architectures, the maximum vscale is almost certainly larger than 1.
> 
> Regardless, I would think that it should be none by default. It's an unknowable runtime constant after all.
Agreed, None should be the default.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93030