[PATCH] D27518: Moving isComplex decision to TTI

Elena Demikhovsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 13 09:11:15 PST 2016


delena added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:606
+  /// of address-computation cost 
+  struct AddressAccessInfo {
+    bool  isStrided;        /// True in case the access is strided (AddRec).
----------------
Why do you need a struct? "Value *" will contain everything.

Value *Stride;

if (Stride == nullptr) - no stride,
dyn_cast<ConstantInt>(Stride) - answers the question isConstant()
 


https://reviews.llvm.org/D27518





More information about the llvm-commits mailing list