[PATCH] D89872: [SVE][AArch64] Fix TypeSize warning in GEP cost analysis

Sander de Smalen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 21 09:18:02 PDT 2020


sdesmalen added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:785
     bool HasBaseReg = (BaseGV == nullptr);
 
     auto PtrSizeBits = DL.getPointerTypeSizeInBits(Ptr->getType());
----------------
If any of the indices is a scalable vector, then the result type will also be a scalable vector. So you can bail out at the start of the function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89872



More information about the llvm-commits mailing list