[PATCH] D75525: [TTI][ARM][MVE] Refine gather/scatter cost model

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.
Herald added a subscriber: wuzish.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:968
   ///                   that is not a compile-time constant
   /// \p Alignment - alignment of single element
   int getGatherScatterOpCost(unsigned Opcode, Type *DataTy, Value *Ptr,
----------------
Please can you add the doxygen param entry?


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:1009
   /// Three cases are handled: 1. scalar instruction 2. vector instruction
   /// 3. scalar instruction which is to be vectorized with VF.
   int getIntrinsicInstrCost(Intrinsic::ID ID, Type *RetTy,
----------------
Explain what I is


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

https://reviews.llvm.org/D75525