[PATCH] D92985: [SystemZTTIImpl::getMinPrefetchStride] Allow some non-prefetched mem accesses.

[Ulrich Weigand via Phabricator via llvm-commits]

uweigand added a comment.

> It doesn't seem right to bail just for one non-prefetched access, so it seems reasonable to allow a relatively very small amount of non-prefetched instructions, to make the heuristic more stable. This patch suggests allowing 1 non-prefetched memory access per 32 prefetched ones. This handles LBM, and also gives two prefetches to imagick which however do not seem to play a role.

This sounds reasonable to me, but this is the kind of question that in the end only measurement can decide ...   What is the overall effect of this patch on benchmarks?  Anything else beyond lbm and imagick?


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

https://reviews.llvm.org/D92985