[PATCH] D90950: [AArch64]Add memory op cost model for SVE

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 13:44:14 PST 2020


sdesmalen accepted this revision.
sdesmalen added a comment.

LGTM, thanks for the changes @CarolineConcatto



================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:791
       ProfitableNumElements = 8;
-
     if (cast<FixedVectorType>(Ty)->getNumElements() < ProfitableNumElements) {
----------------
nit: unrelated change.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90950



More information about the llvm-commits mailing list