[PATCH] D130418: [LoopDataPrefetch] Fix crash when TTI doesn't set CacheLineSize

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 25 04:22:12 PDT 2022


Allen added a comment.

In D130418#3675649 <https://reviews.llvm.org/D130418#3675649>, @tschuett wrote:

> Do you force all targets to provide `PrefetchDistance` and `PrefetchDistance` or could it be an `llvm::optional` ?

Thanks, I think it is unfriendly to crash at the moment. Use a default value when they are not specified can be independent next step, if you feel the need.


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

https://reviews.llvm.org/D130418



More information about the llvm-commits mailing list