[PATCH] D63614: [System Model] [TTI] Update cache and prefetch TTI interfaces

[David Greene via Phabricator via llvm-commits]

greened marked an inline comment as done.
greened added inline comments.
Herald added subscribers: wuzish, MaskRay.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:390
 
-  unsigned getMinPrefetchStride() { return 1; }
+  /// \return The minimum stride in bytes where it makes sense to
+  /// start adding SW prefetches.  Some HW prefetchers can handle
----------------
hfinkel wrote:
> I don't see any value in repeating these interface comments here. They're already present on the declarations in TargetTransformInfo.h, and they'll get out of sync here.
Ok.  I wasn't sure what the policy is.  Some implementations have them, others don't.  I agree that less duplication is better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63614